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

Use autoconf to generate Makevars at install time #787

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

quinn-r88
Copy link

This PR adds a new configure script to the package, which will generate Makevar files when the package is installed. The configure script is generated by GNU autoconf from configure.ac, and must be regenerated by the developer when configure.ac is modified.

This PR also adds a cleanup script, which R will use to remove the files that configure will generate.

Resolves #786

References:

@Jean-Romain
Copy link
Collaborator

Jean-Romain commented Nov 18, 2024

The PR is failing on windows. I know the error, I got it in another package and I remember I had a lot of trouble figuring out how to fix it. Sadly I do not remember the procedure.

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.

Cannot install lidR on R 3.6.3 due to cpp 14+ requirements
2 participants