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

Expand docs, dump config.log on failure, version bump #19

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

Conversation

dubiousjim
Copy link
Contributor

Made an attempt to document the changes I pushed and you merged recently. Also to give some context about the difference between Rust's (and this package's) interpretation of "host/target" and that of Gnu autotools.

When developing my own package which used this one, I found it helpful to see a dump of config.log if the configure script failed. Proposing that change here too. Am glad to revert it or make it a separate PR if you like.

Finally, bumped the version. Please consider publishing the new version, so that I can publish my own package that relies on these changes.

@dubiousjim
Copy link
Contributor Author

ping

@lu-zero
Copy link
Owner

lu-zero commented May 7, 2021

Somehow the notification for this got lost :/

@lu-zero
Copy link
Owner

lu-zero commented May 7, 2021

I took in the documentation changes, using cat is not going to work on certain targets, but printing on stderr where config.log is might be a good idea.

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.

None yet

2 participants