You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@couchdb.apache.org by "nickva (via GitHub)" <gi...@apache.org> on 2024/09/17 13:44:46 UTC
[PR] Don't fail make clean if hex is not installed [couchdb]
nickva opened a new pull request, #5250:
URL: https://github.com/apache/couchdb/pull/5250
`@mix clean --deps` apparently requires hex to be installed. If it's not there, it stops and prompts the user to install it. If configure hasn't run yet, as in the case when building our deb package from the dist tarball, package build fails.
This was mitigated in couchdb-pkg by adding hex to `/home/jenkins/.mix` for CI images, but let's also handle it better in the Makefile.
Makefile.win already has `|| true` added to it so no need to updated it.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@couchdb.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Don't fail make clean if hex is not installed [couchdb]
Posted by "nickva (via GitHub)" <gi...@apache.org>.
nickva commented on PR #5250:
URL: https://github.com/apache/couchdb/pull/5250#issuecomment-2356293720
@jaydoane no problem at all. It was a tricky one as it was failing with dist tarball on my docker instance. I remember we tried to use the full CI build to validate this case and it passed there. But I think in the CI jenkins instance we have a mounted shared mix archive cache so it was finding the hex package but on my docker it couldn't.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@couchdb.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Don't fail make clean if hex is not installed [couchdb]
Posted by "nickva (via GitHub)" <gi...@apache.org>.
nickva merged PR #5250:
URL: https://github.com/apache/couchdb/pull/5250
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@couchdb.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Don't fail make clean if hex is not installed [couchdb]
Posted by "jaydoane (via GitHub)" <gi...@apache.org>.
jaydoane commented on PR #5250:
URL: https://github.com/apache/couchdb/pull/5250#issuecomment-2356222279
Thanks for figuring out why this was failing and how to properly fix it Nick!
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@couchdb.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org