Page MenuHomeSolus

Update graph-tool to version 2.29
ClosedPublic

Authored by padraig on Aug 6 2019, 4:37 PM.
Tags
None
Referenced Files
F11052240: D6899.id.diff
Thu, Aug 10, 9:09 PM
F11052239: D6899.id16536.diff
Thu, Aug 10, 9:09 PM
F11052238: D6899.id16587.diff
Thu, Aug 10, 9:09 PM
F11052237: D6899.id17216.diff
Thu, Aug 10, 9:09 PM
F11052236: D6899.id17148.diff
Thu, Aug 10, 9:09 PM
F11052235: D6899.id16529.diff
Thu, Aug 10, 9:09 PM
F11052234: D6899.id17147.diff
Thu, Aug 10, 9:09 PM
F11052233: D6899.id16528.diff
Thu, Aug 10, 9:09 PM

Details

Summary

Changelog available here
Fixes T8229

Test Plan
$ python3
>>> import graph_tool.all as gt
>>> g = gt.Graph()
>>> g.add_edge(0,1)
>>> gt.interactive_window(g)

Diff Detail

Repository
R4213 graph-tool
Branch
master
Lint
No Lint Coverage
Unit
No Test Coverage

Event Timeline

padraig created this revision.
padraig retitled this revision from Update to version 2.29 to fix T8229 Update flags to reduce memory usage during the build to Update to version 2.29 to fix T8229.
padraig edited the summary of this revision. (Show Details)
padraig edited the test plan for this revision. (Show Details)

Fixed test plan

Jacalz added inline comments.
package.yml
6

License should be switched back to GPL-3.0-or-later.

13

You can remove the extra space from here ;)

20

You can add matplotlib back to it’s original place so the list is ordered alphabetically again.

26

Is there anything wrong with the %configure macro here?

kyrios123 retitled this revision from Update to version 2.29 to fix T8229 to Update graph-tool to version 2.29.Aug 6 2019, 5:23 PM
kyrios123 edited the summary of this revision. (Show Details)
kyrios123 edited the test plan for this revision. (Show Details)

In case you're wondering why these changes crept in it's cause I had an older build on my computer which I used instead of going from the most recent version, not very used to doing this, hopefully it's okay now.

JoshStrobl added a subscriber: JoshStrobl.
JoshStrobl added inline comments.
pspec_x86_64.xml
309–354

None of this should be in /usr/local. Try adding PREFIX=/usr before the PYTHON bit on the configure line.

This revision now requires changes to proceed.Aug 8 2019, 3:46 PM
padraig marked 4 inline comments as done.
  • Rebuilt with PREFIX=/usr
This revision is now accepted and ready to land.Aug 16 2019, 9:35 AM

Your patch does not apply cleanly on top of master, have you been amending your git commits or making new ones?

This revision now requires changes to proceed.Aug 16 2019, 9:39 AM

I've been ammending them. Should I just do the most recent as a commit? Apologies, I'm not used to using git so was just trying to follow the documentation.

Restarted process and rebuilt

LGTM, let's see if it cleanly applies now.

This revision is now accepted and ready to land.Sep 17 2019, 10:47 AM