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

build: resolve aconfigure location #3691

Closed
wants to merge 1 commit into from

Conversation

gjasny
Copy link
Contributor

@gjasny gjasny commented Sep 11, 2023

Come package managers like Conan prefer to use configure over aconfigure. Doing so right now fails if the working directory is not in the root of the source tree.

@nanangizz
Copy link
Member

I am not familiar with Conan, just curious, why not change the working directory first before invoking configure as instructed by the docs: cd pjproject then ./configure? The aconfigure script itself also seems to require to be executed from the root of the source tree (e.g: ac_pjdir is set using pwd here).

@sauwming
Copy link
Member

sauwming commented Oct 9, 2023

Closing this due to lack of follow-up/update. Please reopen if there's more information.

@sauwming sauwming closed this Oct 9, 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