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 build on GNU/Hurd #9514

Closed
wants to merge 1 commit into from
Closed

Fix build on GNU/Hurd #9514

wants to merge 1 commit into from

Conversation

sthibaul
Copy link

There is no path length limitation there, even via pathconf. But glibc
provides a getcwd function that allocates the buffer dynamically so we can
just leverage that.

There is no path length limitation there, even via pathconf.  But glibc
provides a getcwd function that allocates the buffer dynamically so we can
just leverage that.
@albert-github
Copy link
Collaborator

Thanks for pointing this out, though the file filesystem.hpp is an imported function from the https://github.com/gulrak/filesystem project (i.e. the file: https://github.com/gulrak/filesystem/blob/master/include/ghc/filesystem.hpp), so the problem should be reported upstream at the filseystem project as well.

@sthibaul
Copy link
Author

Ok, submitted on gulrak/filesystem#154

@albert-github
Copy link
Collaborator

I see that the fix gulrak/filesystem#154 has been integrated in the master version on https://github.com/gulrak/
I just created an issue about the plans for a new release gulrak/filesystem#164

@albert-github
Copy link
Collaborator

albert-github commented Apr 18, 2023

I see that the new release of filesystem is out for some time and that it has been integrated in the doxygen master as well.
So do you still have the same problems when using the master version of doxygen or has this been solved now?
Actually I think this PR can be closed as fixed upstream.

@albert-github albert-github added the fixed but not released Bug is fixed in github, but still needs to make its way to an official release label Apr 25, 2023
@doxygen
Copy link
Owner

doxygen commented May 18, 2023

This issue was previously marked 'fixed but not released',
which means it should be fixed in doxygen version 1.9.7.
Please verify if this is indeed the case. Reopen the
issue if you think it is not fixed and please include any additional information
that you think can be relevant (preferably in the form of a self-contained example).

@doxygen doxygen removed the fixed but not released Bug is fixed in github, but still needs to make its way to an official release label May 18, 2023
@doxygen doxygen closed this May 18, 2023
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.

3 participants