Page MenuHomeSolus

Update graph-tool to 2.35 and fix T9215
ClosedPublic

Authored by padraig on Aug 6 2020, 2:25 PM.
Tags
None
Referenced Files
F11019272: D9382.diff
Sun, Aug 6, 6:43 PM
F10997212: D9382.id23377.diff
Thu, Jul 27, 4:35 AM
F10958444: D9382.id23458.diff
Tue, Jul 18, 12:54 AM
F10954667: D9382.id22639.diff
Mon, Jul 17, 3:16 AM
F10845302: D9382.id23458.diff
Jun 10 2023, 11:46 PM
F10824105: D9382.diff
Jun 3 2023, 10:48 AM
F10780061: D9382.id22640.diff
May 24 2023, 10:14 PM
F10772514: D9382.id.diff
May 21 2023, 1:15 PM

Details

Summary

Rebuild graph-tool against libboost
Depends on D8527

Update graph-tool to 2.35 and fix T9215

Updates to newest version of graph-tool but does not compile with clang as this causes errors when importing submodules.

Test Plan

Rebuild it against libboost

import graph_tool.correlations

If no error then submodules load okay.

Diff Detail

Repository
R4213 graph-tool
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

padraig created this revision.
Girtablulu added a subscriber: Girtablulu.

please rebase on the correct release, did you forget to make pull before updating it?

This revision now requires changes to proceed.Aug 6 2020, 3:04 PM

Damn it! This is what I thought I did between the last attempt and this one but clearly messed it up somehow.... I'll try again, do I need to build it each time or can I just use these files after making a pull?

you need to rebase, so you will kill these changes because 99% is based on version 9, after rebase you need to run make pull to update it to the latest version, update it to the newest version, build it and update this commit via arc diff -update D9382

  • Update graph-tool to 2.33 and fix T9215

Rebased

you need to rebase, so you will kill these changes because 99% is based on version 9, after rebase you need to run make pull to update it to the latest version, update it to the newest version, build it and update this commit via arc diff -update D9382

Apologies for being so useless here, I'm very inexperienced with github, before I saw your message I cloned it again and then did git config pull.rebase true, then make and updated with acc diff, however it still seems to be on version 8. I did make pull now so hopefully after I've rebuilt it for the 10th time for my sins this will work. It'll take a few hours before I can update it again though.

after rebase and make pull check your package.yml if it's updated, or just reclone the repo add your changes rebuild it and run arc diff --update D9382 which should work as well

I think it worked finally, sorry about that. Note that I build it with -j2 for memory purposes, that might need to be removed or maybe changed to -j3.

looks good :) but please remove the -j2 from the package.yml, the buildserver can handle this. you dont need to rebuild it just git commit --ammend it and arc diff it

Should be good now, thanks for your help!

Hi! This should be merged as part of the LLVM upgrade. Could you update this to version 2.35? Would be appreciated.

This revision now requires changes to proceed.Sep 22 2020, 2:08 PM

This is now updated to version 2.35

Summary and title should be updated.

serebit retitled this revision from Update graph-tool to 2.33 and fix T9215 to Update graph-tool to 2.35 and fix T9215.
serebit edited the summary of this revision. (Show Details)
This revision now requires changes to proceed.Sep 22 2020, 2:45 PM
This revision now requires review to proceed.Sep 22 2020, 2:45 PM

Sorry for that. Should be good to go now, @JoshStrobl.

This revision was not accepted when it landed; it landed in state Needs Review.Sep 28 2020, 8:07 PM
This revision was automatically updated to reflect the committed changes.