-
Notifications
You must be signed in to change notification settings - Fork 2k
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
dev-cpp/ETL: EAPI7, add missing die, use HTTPs #9031
Conversation
Ping, just rebased this PR, any chance for a merge? |
@lu-zero: ping? |
dev-cpp/ETL/ETL-0.04.15.ebuild
Outdated
|
||
inherit autotools | ||
|
||
DESCRIPTION="ETL is a multi-platform class and template library" | ||
HOMEPAGE="http://synfig.org" | ||
HOMEPAGE="https://www.synfig.org" | ||
SRC_URI="mirror://sourceforge/synfig/${PN}/${PV}/${P}.tar.gz" | ||
|
||
LICENSE="GPL-2" | ||
SLOT="0" | ||
KEYWORDS="~amd64 ~x86" | ||
IUSE="" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
empty IUSE may be deleted
ping, just rebased this PR |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hmm... In-place EAPI bumps aren't recommended and should happen in a new, separate file.
@monsieurp |
ping, any chance we can merge this :) |
Closes: https://bugs.gentoo.org/663798 Signed-off-by: Michael Mair-Keimberger <m.mairkeimberger@gmail.com>
another ping, would be nice if we could merge this :) |
Pull request CI reportReport generated at: 2019-02-08 18:00 UTC No issues found |
Hi,
This PR raises EAPI, adds a missing
die
and uses https instead of http. Also dropped the *DEPEND vars.I didn't introduced a new revision as nothing changed installation wise and the package isn't stable anyway.
Please review.