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
centos8 #636
centos8 #636
Conversation
(cherry picked from commit d5f9715)
RUN ./deps/readies/bin/getpy2 | ||
RUN test -f /usr/bin/python || ln -s /usr/bin/python2 /usr/bin/python |
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.
Also possible: FIX=1 ./deps/readies/bin/getpy2
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.
We're not merging this back to master. It's a cut-throat patch. Didn't know, thanks - will use next time!
@@ -117,7 +117,9 @@ OBJECTS=$(patsubst $(SRCDIR)/%.c,$(BINDIR)/%.o,$(SOURCES)) | |||
CC_DEPS = $(patsubst $(SRCDIR)/%.c, $(BINDIR)/%.d, $(SOURCES)) | |||
|
|||
CC_FLAGS += \ | |||
-fPIC -std=gnu99 \ | |||
-fPIC \ | |||
-fcommon \ |
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.
IMO this is more risky than my change, but I'll go with it.
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.
@rafie can you explain why? What risks do you see with it?
(cherry picked from commit 883918f)
No description provided.