update script in ubuntu-meta not handling germinate or specified versions

Bug #2017786 reported by Tim Andersson
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
edubuntu-meta (Ubuntu)
Won't Fix
Undecided
Unassigned
kubuntu-meta (Ubuntu)
New
Undecided
Unassigned
lubuntu-meta (Ubuntu)
New
Undecided
Unassigned
ubuntu-budgie-meta (Ubuntu)
New
Medium
fossfreedom
ubuntu-mate-meta (Ubuntu)
New
Undecided
Unassigned
ubuntu-meta (Ubuntu)
Fix Released
Medium
Tim Andersson
ubuntu-unity-meta (Ubuntu)
New
Undecided
Unassigned
ubuntucinnamon-meta (Ubuntu)
New
Undecided
Joshua Peisach
ubuntukylin-meta (Ubuntu)
New
Undecided
handsome_feng
ubuntustudio-meta (Ubuntu)
Won't Fix
Undecided
Unassigned
xubuntu-meta (Ubuntu)
New
Undecided
Unassigned

Bug Description

update script handled debootstrap and devscripts, does not handle germinate. also doesn't handle potential specified versions in the package directory.

Tags: patch
Revision history for this message
Ubuntu Foundations Team Bug Bot (crichton) wrote :

The attachment "mysuperpatch.debdiff" seems to be a debdiff. The ubuntu-sponsors team has been subscribed to the bug report so that they can review and hopefully sponsor the debdiff. If the attachment isn't a patch, please remove the "patch" flag from the attachment, remove the "patch" tag, and if you are member of the ~ubuntu-sponsors, unsubscribe the team.

[This is an automated message performed by a Launchpad user owned by ~brian-murray, for any issue please contact him.]

tags: added: patch
Revision history for this message
Graham Inggs (ginggs) wrote :

The debdiff is against 1.481 in jammy, please rebase against the latest version in the development release (currently 1.501 in lunar). You can use 'pull-lp-source ubuntu-meta' for this.
Please also close this bug in the changelog e.g. (LP: #2017786).

We should probably do the same for all the flavour meta packages as well, so this bug should be marked as affecting ubuntu-unity-meta, ubuntucinnamon-meta, etc.

Unsubscribing sponsors for now.

Changed in ubuntu-meta (Ubuntu):
importance: Undecided → Medium
status: New → In Progress
assignee: nobody → Tim Andersson (andersson123)
Revision history for this message
Tim Andersson (andersson123) wrote :

Please find attached updated debdiff

Revision history for this message
Tim Andersson (andersson123) wrote :

Okay, ?th time lucky

Changed in ubuntucinnamon-meta (Ubuntu):
assignee: nobody → Joshua Peisach (itzswirlz)
Changed in ubuntustudio-meta (Ubuntu):
assignee: nobody → Erich Eickmeyer (eeickmeyer)
importance: Undecided → Medium
Changed in edubuntu-meta (Ubuntu):
assignee: nobody → Erich Eickmeyer (eeickmeyer)
importance: Undecided → Medium
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package ubuntu-meta - 1.502

---------------
ubuntu-meta (1.502) mantic; urgency=medium

  * Fix update script to include germinate in 'dependency' check and
    refactor script LP: #2017786

 -- Tim Andersson <email address hidden> Fri, 28 Apr 2023 09:09:28 +0100

Changed in ubuntu-meta (Ubuntu):
status: In Progress → Fix Released
Changed in ubuntu-budgie-meta (Ubuntu):
assignee: nobody → fossfreedom (fossfreedom)
importance: Undecided → Medium
Revision history for this message
Steve Langasek (vorlon) wrote :

I do not see what this change was supposed to solve. I disagree with the *existing* version check in germinate, and do not want to see more of these - it imposes needless barriers for updating the metapackages.

Revision history for this message
Erich Eickmeyer (eeickmeyer) wrote (last edit ):

Now that I've had some time to look at the patch, I really don't see it solving anything. After seeing Steve's comment, I do agree, it simply puts up needless barriers to updating metapackages. Sorry, but from Ubuntu Studio and Edubuntu, this is a Nack. I hope other flavors follow suit and don't accept this patch.

Changed in ubuntustudio-meta (Ubuntu):
assignee: Erich Eickmeyer (eeickmeyer) → nobody
importance: Medium → Undecided
status: New → Won't Fix
Changed in edubuntu-meta (Ubuntu):
assignee: Erich Eickmeyer (eeickmeyer) → nobody
importance: Medium → Undecided
status: New → Won't Fix
Changed in ubuntukylin-meta (Ubuntu):
assignee: nobody → handsome_feng (feng-kylin)
Revision history for this message
Brian Murray (brian-murray) wrote :

@eeickmeyer - the version of the change that actually landed in the package essentially adds the following to the update script:

+if ! which germinate-update-metapackage >/dev/null; then
+ echo >&2 "please install germinate"
+ exit 1
+fi

I think this is a useful improvement as not everyone has germinate installed but do as you wish with your packages.

Revision history for this message
Erich Eickmeyer (eeickmeyer) wrote :

@brian-murray

Agreed, simply checking to see whether or not germinate is installed or not is a good idea. However, checking which version is installed, as is currently done with debootstrap, is a bit overkill and I disagree with the debootstrap version check to begin with.

Realistically, if one is running the update script on the current LTS or higher, it shouldn't cause problems and the version check seems to be at least encouraging the update to run on the development version. This seems odd as germinate is pulling from whatever is configured in update.cfg regardless of what series it's being run on, so the version checks just seem like, as Steve put it, needless barriers.

So, in summary, your check for germinate seems sane. A wholesale version check seems overkill and unnecessary.

To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.