-
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
games-emulation/pcsx2: append -mxsave compiler flag with gcc >= 8.2 #11915
Conversation
5bf6987
to
38b6c02
Compare
Closes: https://bugs.gentoo.org/685156 Closes: gentoo#11915 Signed-off-by: Yuri Konotopov <ykonotopov@gnome.org>
Copyright policy changePlease note that on 2018-09-15 Trustees have approved new Gentoo copyright policy. All contributions made to Gentoo need to follow this policy. If you include the Signed-off-by line in your commit message, you indicate that you have read the policy and agree to its terms. For more detailed explanation, please see the new Gentoo copyright policy explained article. Pull Request assignmentSubmitter: @nE0sIghT games-emulation/pcsx2: @nE0sIghT, @gentoo/proxy-maint Linked bugsBugs linked: 685156 In order to force reassignment and/or bug reference scan, please append Docs: Code of Conduct ● Copyright policy (expl.) ● Devmanual ● GitHub PRs ● Proxy-maint guide |
38b6c02
to
424f34f
Compare
Closes: https://bugs.gentoo.org/685156 Closes: gentoo#11915 Signed-off-by: Yuri Konotopov <ykonotopov@gnome.org>
@@ -42,9 +42,15 @@ DEPEND="${RDEPEND} | |||
|
|||
pkg_setup() { | |||
if [[ ${MERGE_TYPE} != binary && $(tc-getCC) == *gcc* ]]; then | |||
if [[ $(gcc-major-version) -lt 4 || $(gcc-major-version) == 4 && $(gcc-minor-version) -lt 8 ]] ; then |
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.
please keep changes limited to the topic of git summary
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.
I splitted commits
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.
It's not a typo; I'd rather drop the whole check - all GCC versions up to 5.5.0 are masked at this point, thus not supported.
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.
It's typo - look one line up. There is no whitespace.
Since there is (masked) versions of gcc in portage I think this check is useful.
However if you insist I will drop this check - one space just doesn't worth time we spent on it.
424f34f
to
4f58c34
Compare
Closes: https://bugs.gentoo.org/685156 Closes: gentoo#11915 Signed-off-by: Yuri Konotopov <ykonotopov@gnome.org>
Closes: https://bugs.gentoo.org/685156 Closes: gentoo#11915 Signed-off-by: Yuri Konotopov <ykonotopov@gnome.org>
4f58c34
to
a101a46
Compare
Pull request CI reportReport generated at: 2019-05-06 15:36 UTC No issues found |
Closes: https://bugs.gentoo.org/685156
Signed-off-by: Yuri Konotopov ykonotopov@gnome.org