Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix hostchk.sh #948

Merged
merged 2 commits into from
Aug 29, 2024
Merged

Fix hostchk.sh #948

merged 2 commits into from
Aug 29, 2024

Conversation

xexyl
Copy link
Contributor

@xexyl xexyl commented Aug 28, 2024

With the recent updates to the inclusion of the different repos and the MKIOCCCENTRY_SRC a problem in linux for hostchk.sh had to be resolved. This might also be a problem in macOS but since the libraries were already installed this is not detected (it seems that in linux if the other libraries were installed this also was not a problem).

There still exists an issue with building, however, as the make dependencies have system paths (cc -MM will solve this so it might be the way independ works but this is not clear to me at this time). Nonetheless the hostchk.sh should at least now work.

New version of hostchk.sh is 1.0.2 2024-08-28.

With the recent updates to the inclusion of the different repos and the
MKIOCCCENTRY_SRC a problem in linux for hostchk.sh had to be resolved.
This might also be a problem in macOS but since the libraries were
already installed this is not detected (it seems that in linux if the
other libraries were installed this also was not a problem).

There still exists an issue with building, however, as the make
dependencies have system paths (cc -MM will solve this so it might be
the way independ works but this is not clear to me at this time).
Nonetheless the hostchk.sh should at least now work.

New version of hostchk.sh is 1.0.2 2024-08-28.
@xexyl
Copy link
Contributor Author

xexyl commented Aug 28, 2024

As noted build will still fail but I believe this to be a problem with the make dependencies. Maybe that tool you wrote can skip over system paths like cc -MM does? Or perhaps it's because of how #ifdef..#endif works?

@lcn2
Copy link
Contributor

lcn2 commented Aug 29, 2024

We had to resolve the conflict and edit the grep regex by hand .. hope this works.

@lcn2 lcn2 merged commit b3e8459 into ioccc-src:master Aug 29, 2024
0 of 2 checks passed
@lcn2
Copy link
Contributor

lcn2 commented Aug 29, 2024

Stay tuned for a fix to Makefiles for dbg, dyn_array, and perhaps jparse too.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants