Opened 10 years ago

Closed 10 years ago

#13415 closed enhancement (fixed)

Prevent unneeded rebuilding when upgrading

Reported by: jdemeyer Owned by: GeorgSWeber
Priority: major Milestone: sage-5.4
Component: build Keywords:
Cc: jhpalmieri Merged in: sage-5.4.beta2
Authors: Jeroen Demeyer Reviewers: John Palmieri
Report Upstream: N/A Work issues:
Branch: Commit:
Dependencies: #13395, #13392, #13416, #12994, #9906, #12883, #13123 Stopgaps:

Status badges

Description (last modified by jdemeyer)

Since (almost) every package depends on patch and bzip2, an update of either of these packages causes a rebuild of everything when upgrading.

An update of prereq even causes everything to be built twice(!) when upgrading (the new prereq script is only seen after the new root repo is installed, so the second sage-upgrade run for double-checking will install the new prereq and therefore rebuild everything).

We should simply remove dependencies of packages on PREREQ, BZIP2 and PATCH and build those packages separately.

Apply to the SAGE_ROOT repository:

  1. 13415_separate_base_deps_auto.patch -- automatically generated by
    sed -i 's| *\$(BASE)||; s| *\$(INST)/\$(PATCH)||' spkg/standard/deps
    
  2. 13415_separate_base_deps_2.patch

Attachments (3)

13415_separate_base_deps_2.patch (9.2 KB) - added by jdemeyer 10 years ago.
13415_separate_base_deps_auto.patch (22.0 KB) - added by jdemeyer 10 years ago.
13415_separate_base_deps_auto_without_12883.patch (21.6 KB) - added by jdemeyer 10 years ago.

Download all attachments as: .zip

Change History (21)

comment:1 Changed 10 years ago by jdemeyer

  • Dependencies changed from #13395 to #13395, #13392

comment:2 Changed 10 years ago by jdemeyer

  • Description modified (diff)

comment:3 Changed 10 years ago by jdemeyer

  • Authors set to Jeroen Demeyer
  • Dependencies changed from #13395, #13392 to #13395, #13392, #13416

comment:4 Changed 10 years ago by jhpalmieri

  • Cc jhpalmieri added

comment:5 Changed 10 years ago by jdemeyer

  • Description modified (diff)

comment:6 Changed 10 years ago by jdemeyer

  • Dependencies changed from #13395, #13392, #13416 to #13395, #13392, #13416, #12994

comment:7 Changed 10 years ago by jdemeyer

  • Dependencies changed from #13395, #13392, #13416, #12994 to #13395, #13392, #13416, #12994, #9906

comment:8 Changed 10 years ago by jdemeyer

John, do you agree in principle with these patches? They are sort-of ready, but I still have to test things properly (including upgrading).

comment:9 Changed 10 years ago by jhpalmieri

I've only given the patches a quick look, but this looks like a good idea (especially after trying an update recently which updated prereq).

comment:10 Changed 10 years ago by jdemeyer

  • Dependencies changed from #13395, #13392, #13416, #12994, #9906 to #13395, #13392, #13416, #12994, #9906, #12883

comment:11 Changed 10 years ago by jdemeyer

  • Dependencies changed from #13395, #13392, #13416, #12994, #9906, #12883 to #13395, #13392, #13416, #12994, #9906, #12883, #13123

Changed 10 years ago by jdemeyer

comment:12 Changed 10 years ago by jdemeyer

  • Status changed from new to needs_review

Changed 10 years ago by jdemeyer

comment:13 Changed 10 years ago by jdemeyer

John, any chance you could formally review this? I have tested this many times on the buildbot (together with other tickets) and it seems to work fine.

comment:14 Changed 10 years ago by kcrisman

This will conflict (not much) with the changes at #10508, which basically has positive review...

comment:15 follow-up: Changed 10 years ago by jhpalmieri

  • Reviewers set to John Palmieri
  • Status changed from needs_review to positive_review

This looks okay to me. I think the conflicts will be easy to resolve.

comment:16 in reply to: ↑ 15 Changed 10 years ago by jdemeyer

Replying to jhpalmieri:

This looks okay to me. I think the conflicts will be easy to resolve.

Indeed, it's just a matter of regenerating the auto-generated patch.

comment:17 Changed 10 years ago by jdemeyer

Many thanks for the review!

comment:18 Changed 10 years ago by jdemeyer

  • Merged in set to sage-5.4.beta2
  • Resolution set to fixed
  • Status changed from positive_review to closed
Note: See TracTickets for help on using tickets.