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

New package: libsecp256k1-20190311+ee99f12. #10176

Closed
wants to merge 1 commit into from
Closed

New package: libsecp256k1-20190311+ee99f12. #10176

wants to merge 1 commit into from

Conversation

DBLouis
Copy link
Contributor

@DBLouis DBLouis commented Mar 23, 2019

No description provided.

}

do_build() {
make -j${XBPS_MAKEJOBS}
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Use build_style=gnu-configure instead of manually calling make in build and install

checksum=e4939619f89ca48da1dd5f80435fe5433be9b98b3377cb9d67d1f1e5527db05e

do_configure() {
./autogen.sh
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

do autoreconf -if in pre_configure instead

pkgname=libsecp256k1
version=20190311+ee99f12
revision=1
_commit=ee99f12f3d8af18bbb9b74cd31907ac1979742ec
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please ask upstream for a tagged release, I doubt this will be merged otherwise

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

revision=1
_commit=ee99f12f3d8af18bbb9b74cd31907ac1979742ec
wrksrc="secp256k1-${_commit}"
makedepends="automake autoconf libtool"
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

These should be hostmakedepends and autoconf isn't required.

Copy link
Contributor

@maxice8 maxice8 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Requires upstream tagged release. They don't consider it ready.

@maxice8 maxice8 closed this Apr 3, 2019
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