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

Potential issue in polybori - 0.5rc.p8 and/or 0.5rc.p9 #6582

Closed
sagetrac-drkirkby mannequin opened this issue Jul 21, 2009 · 10 comments
Closed

Potential issue in polybori - 0.5rc.p8 and/or 0.5rc.p9 #6582

sagetrac-drkirkby mannequin opened this issue Jul 21, 2009 · 10 comments

Comments

@sagetrac-drkirkby
Copy link
Mannequin

sagetrac-drkirkby mannequin commented Jul 21, 2009

I believe there is an issue which may affect Solaris with polybori 0.5rc.p8, and assuming my patch to .p9 gets positive review, will affect that too, as I have not tried to fix this.

Here are some notes I put in patches/custom.py

# Note, these 'SAGE_DEBUG' linker flags added by someone
# are likely to break if used on Solaris
# with the Sun linker, as -p option to the Sun linker is:
#         [-p auditlib]   identify audit library to accompany this object
# This has not been confirmed, and I don't have time to test it.
# David Kirkby, 21st July 2009. I suggest this is revisited by someone soon.
if os.environ.has_key('SAGE_DEBUG'):
    CPPDEFINES=[]
    CCFLAGS=[" -pg"] + CCFLAGS
    CXXFLAGS=[" -pg"] + CXXFLAGS
    LINKFLAGS=[" -pg"]



Duplicate of #11575.

Upstream: Completely fixed; Fix reported upstream

CC: @alexanderdreyer @sagetrac-PolyBoRi @sagetrac-drkirkby

Component: porting: Solaris

Reviewer: Alexander Dreyer

Issue created by migration from https://trac.sagemath.org/ticket/6582

@sagetrac-drkirkby sagetrac-drkirkby mannequin added this to the sage-4.1.2 milestone Jul 21, 2009
@sagetrac-drkirkby
Copy link
Mannequin Author

sagetrac-drkirkby mannequin commented Sep 28, 2009

comment:1

Another issue, which is this case I am 100% sure about, is that PolyBoRi (as of polybori-0.6.3-20090827.spkg) in sage-4.1.2.alpha4 is that PolyBoRi is sending GNU-specific options to the Sun compiler. See #7034

@sagetrac-drkirkby sagetrac-drkirkby mannequin modified the milestones: sage-4.1.2, sage-4.2 Sep 28, 2009
@jdemeyer
Copy link
Contributor

comment:3

Is this still a problem?

@alexanderdreyer
Copy link
Mannequin

alexanderdreyer mannequin commented Apr 30, 2012

comment:4

No, it was fixed. For instance, in #12655 for PolyBoRi 0.8.1.

@alexanderdreyer
Copy link
Mannequin

alexanderdreyer mannequin commented Apr 30, 2012

Upstream: Completely fixed; Fix reported upstream

@alexanderdreyer
Copy link
Mannequin

alexanderdreyer mannequin commented Jun 25, 2012

comment:5

Duplicate of#12655.

@alexanderdreyer

This comment has been minimized.

@alexanderdreyer
Copy link
Mannequin

alexanderdreyer mannequin commented Jun 25, 2012

comment:6

Abusing "positive review".

@jdemeyer
Copy link
Contributor

Reviewer: Alexander Dreyer

@jdemeyer
Copy link
Contributor

comment:7

In such cases, you should set the milestone to "sage-duplicate/invalid/wontfix".

@jdemeyer jdemeyer removed this from the sage-5.1 milestone Jun 25, 2012
@alexanderdreyer
Copy link
Mannequin

alexanderdreyer mannequin commented Jun 25, 2012

comment:8

Replying to @jdemeyer:

In such cases, you should set the milestone to "sage-duplicate/invalid/wontfix".

Thanks, I'll do so next time.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant