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

Merge ebuilds to reduce maintenance costs. #9

Merged
merged 1 commit into from
Dec 24, 2013
Merged

Merge ebuilds to reduce maintenance costs. #9

merged 1 commit into from
Dec 24, 2013

Conversation

omasanori
Copy link
Contributor

Currently rust-0.8.ebuild and rust-9999.ebuild have some minor
differences. After this change, you'll modify rust-9999.ebuild to fix
build problems in master as usual, and then all you need to support a
new release is just cp rust-9999.ebuild rust-x.y.ebuild; <edit KEYWORDS in rust-x.y.ebuild>.

I don't know why you've avoided using econf, so I left the line
"${ECONF_SOURCE:-.}"/configure.

Currently rust-0.8.ebuild and rust-9999.ebuild have some minor
differences. After this change, you'll modify rust-9999.ebuild to fix
build problems in master as usual, and then all you need to support a
new release is just `cp rust-9999.ebuild rust-x.y.ebuild; <edit KEYWORDS
in rust-x.y.ebuild>`.

I don't know why you've avoided using econf, so I left the line
"${ECONF_SOURCE:-.}"/configure.

Signed-off-by: OGINO Masanori <masanori.ogino@gmail.com>
@cnd
Copy link
Contributor

cnd commented Dec 24, 2013

thanks, I avoid econf because work for it's support is not finished yet.

It (econf) was working on my fork but now I re-based commits to sync with master. I will back to this problem when I will have time.

Maybe after my contribution econf will work for amd64 but for x86 they should have pc-linux instead of unknown-linux in triple.

cnd added a commit that referenced this pull request Dec 24, 2013
Merge ebuilds to reduce maintenance costs.
@cnd cnd merged commit 28c8cc6 into gentoo:master Dec 24, 2013
@omasanori omasanori deleted the unify-ebuild branch December 24, 2013 08:49
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.

2 participants