-
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
net-misc/unison: bump to 2.51.2 #9173
Conversation
This adds support for slotted fsmonitor which will be added in upcoming net-misc/unison ebuilds Package-Manager: Portage-2.3.41, Repoman-2.3.9
Otherwise, we can't install a parallel version (which is upcoming). Also, bump to EAPI 7 and restrict tests (see comment) Package-Manager: Portage-2.3.41, Repoman-2.3.9
Closes: https://bugs.gentoo.org/651972 Package-Manager: Portage-2.3.41, Repoman-2.3.9
net-misc/unison/unison-2.51.2.ebuild
Outdated
|
||
EAPI="7" | ||
|
||
IUSE="gtk doc static debug threads +ocamlopt test" |
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.
at least move it acording to variables order ;)
net-misc/unison/unison-2.51.2.ebuild
Outdated
|| ( net-misc/x11-ssh-askpass net-misc/ssh-askpass-fullscreen ) ) | ||
>=app-eselect/eselect-unison-0.4" | ||
|
||
SRC_URI="https://www.seas.upenn.edu/~bcpierce/unison/download/releases/${P}/${P}.tar.gz |
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.
this too
Pull request CI report Report generated at: 2018-07-12 12:54 UTC No issues found |
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.
Just one minor comment, otherwise LGTM. Please go ahead and commit it.
dodir ${MODULEDIR} | ||
insinto ${MODULEDIR} | ||
newins "${FILESDIR}/${MODULE}.eselect-${PVR}" ${MODULE}.eselect \ | ||
|| die "failed to install" |
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.
You don't need || die
for ebuild helpers like newins
.
Done. |
@heroxbd here it is. I had to bump
eselect-unison
as well as revbump2.48.4
so that it slotsunison-fsmonitor
.