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.

Sign up for GitHub

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

Jump to bottom

Merging fixes for PolyBoRi 0.7.1 into 0.8 spkg #11909

Closed
alexanderdreyer mannequin opened this issue Oct 8, 2011 · 19 comments
Closed

Merging fixes for PolyBoRi 0.7.1 into 0.8 spkg #11909

alexanderdreyer mannequin opened this issue Oct 8, 2011 · 19 comments
Assignees
@alexanderdreyer
Labels
c: algebra r: duplicate t: bug

Comments

@alexanderdreyer
Copy link
Mannequin

alexanderdreyer mannequin commented Oct 8, 2011

After #11575 was positively reviewed the PolyBoRi 0.7.1 spkg was improved in #11906.

So we have to merge the recent changes from http://boxen.math.washington.edu/home/dreyer/spkg/polybori-0.7.1.p7.spkg into http://boxen.math.washington.edu/home/dreyer/spkg/polybori-0.8.0.p1.spkg

Current patches/spkg

Depends on #12750
Depends on #12655

Upstream: Fixed upstream, in a later stable release.

CC: @nexttime

Component: algebra

Reviewer: Alexander Dreyer, Leif Leonhardy

Issue created by migration from https://trac.sagemath.org/ticket/11909
@alexanderdreyer alexanderdreyer mannequin added this to the sage-5.0 milestone Oct 8, 2011
@alexanderdreyer alexanderdreyer mannequin self-assigned this Oct 8, 2011
@alexanderdreyer

This comment has been minimized.

Sign in to view
@alexanderdreyer
Copy link
Mannequin Author

alexanderdreyer mannequin commented Oct 8, 2011

comment:1

The new spkg is here:
http://boxen.math.washington.edu/home/dreyer/spkg/polybori-0.8.0.p2.spkg
It also utilizes __M4RI_CFLAGS when available.

@alexanderdreyer

This comment has been minimized.

Sign in to view
@alexanderdreyer
Copy link
Mannequin Author

alexanderdreyer mannequin commented Oct 9, 2011

comment:3

Due to change in the 0.7.1 spkg, I had to rebundle another spkg (at the same place above).

@alexanderdreyer
Copy link
Mannequin Author

alexanderdreyer mannequin commented Oct 11, 2011

comment:4

The new spkg builds, installs and passes all tests on SuSE Enterprise 11 (with SP1) on amd64 and OS X 10.5 ppc.

@alexanderdreyer

This comment has been minimized.

Sign in to view
@alexanderdreyer
Copy link
Mannequin Author

alexanderdreyer mannequin commented Mar 26, 2012

comment:5

maybe need to merge with the new p2.spkg

@jdemeyer
Copy link
Contributor

jdemeyer commented Apr 2, 2012

comment:6

How is this related to #12750 and #12655?

@nexttime
Copy link
Mannequin

nexttime mannequin commented Apr 2, 2012

Work Issues: Rebase the spkg on the p2 from #12750.

@nexttime
Copy link
Mannequin

nexttime mannequin commented Apr 2, 2012

Changed dependencies from #11575 #11906 to #11575 #11750

@nexttime
Copy link
Mannequin

nexttime mannequin commented Apr 2, 2012

comment:7

Replying to @jdemeyer:

How is this related to #12750 and #12655?

The spkg here should get rebased on that of #12750, to a p3.

If #12655 (currently needing work) gets merged before this one, this ticket can be closed, as upgrading to 0.8.1 would also fix the issue, if I'm not completely wrong.

@nexttime nexttime mannequin added s: needs review and removed s: needs info labels Apr 2, 2012
@nexttime
Copy link
Mannequin

nexttime mannequin commented Apr 2, 2012

Changed author from AlexanderDreyer to Alexander Dreyer

@nexttime
Copy link
Mannequin

nexttime mannequin commented Apr 2, 2012

Changed dependencies from #11575 #11750 to #12750

@nexttime nexttime mannequin added s: needs work and removed s: needs review labels Apr 2, 2012
@nexttime
Copy link
Mannequin

nexttime mannequin commented Apr 9, 2012

comment:9

Replying to @nexttime:

If #12655 (currently needing work) gets merged before this one, this ticket can be closed, as upgrading to 0.8.1 would also fix the issue, if I'm not completely wrong.

So can we close this ticket now?

@nexttime nexttime mannequin removed this from the sage-5.0 milestone Apr 9, 2012
@nexttime nexttime mannequin added s: needs info and removed s: needs work labels Apr 9, 2012
@alexanderdreyer
Copy link
Mannequin Author

alexanderdreyer mannequin commented Apr 10, 2012

comment:10

Replying to @nexttime:

So can we close this ticket now?

Yes, it's obsolete now.

@nexttime
Copy link
Mannequin

nexttime mannequin commented Apr 10, 2012

Reviewer: Alexander Dreyer, Leif Leonhardy

@nexttime
Copy link
Mannequin

nexttime mannequin commented Apr 10, 2012

Changed author from Alexander Dreyer to none

@nexttime nexttime mannequin added s: needs review and removed s: needs info labels Apr 10, 2012
@nexttime
Copy link
Mannequin

nexttime mannequin commented Apr 10, 2012

Changed dependencies from #12750 to #12750 #12655

@nexttime
Copy link
Mannequin

nexttime mannequin commented Apr 10, 2012

Changed work issues from Rebase the spkg on the p2 from #12750. to none

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@alexanderdreyer alexanderdreyer

Labels
c: algebra r: duplicate t: bug
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@jdemeyer