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

freebsd.eclass: Fix for FreeBSD10.0 or later #36

Closed
wants to merge 2 commits into from
Closed

freebsd.eclass: Fix for FreeBSD10.0 or later #36

wants to merge 2 commits into from

Conversation

nigoro
Copy link

@nigoro nigoro commented Feb 22, 2015

-9999 ebuild, changed to use to force /usr/share/mk.
Fixed a problem of numerical comparison in 10.0.

Details please see
https://bugs.gentoo.org/show_bug.cgi?id=488214#c23
https://bugs.gentoo.org/show_bug.cgi?id=488214#c24

-9999 ebuild, changed to use to force /usr/share/mk on FreeBSD 10.0.
Fixed a problem of numerical comparison in 10.0.
@@ -141,7 +147,8 @@ freebsd_src_unpack() {

# Starting from FreeBSD 9.2, its install command supports the -l option and
# they now use it. Emulate it if we are on a system that does not have it.
if [[ ${RV} > 9.1 ]] && ! has_version '>=sys-freebsd/freebsd-ubin-9.2_beta1' ; then
version_compare ${RV} 9.1
Copy link
Member

Choose a reason for hiding this comment

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

Please avoid version_compare since it's hard to read. Prefer ! version_is_at_least.

@nigoro
Copy link
Author

nigoro commented Feb 22, 2015

@mgorny, Thank you for advice. I have updated the patch.

The results of the test are as follows.
9.1 --> False
9.2 --> True
10.1 --> True

@mgorny
Copy link
Member

mgorny commented Feb 22, 2015

+  22 Feb 2015; Michał Górny <mgorny@gentoo.org> freebsd.eclass:
+  Fix support for FreeBSD 10.0. Force /usr/share/mk there, and fix version
+  comparison for install commands.
+  https://github.com/gentoo/gentoo-portage-rsync-mirror/pull/36 by nigoro.

@mgorny mgorny closed this Feb 22, 2015
minaco2 pushed a commit to git-portage/git-portage that referenced this pull request Feb 22, 2015
@nigoro nigoro deleted the bug488214_eclass branch April 18, 2015 07:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants