[Merge] ~vorlon/casper:cleanup into casper:main

Glenn Washburn mp+452496 at code.launchpad.net
Wed Oct 4 19:15:24 UTC 2023


Review: Needs Fixing

Looks like my comment on language-selector.desktop may be irrelevant if its going to be dropped entirely. But there does appear to be an issue in the changelog.

Diff comments:

> diff --git a/debian/changelog b/debian/changelog
> index c131f39..1bdc544 100644
> --- a/debian/changelog
> +++ b/debian/changelog
> @@ -1,8 +1,45 @@
>  casper (1.485) UNRELEASED; urgency=medium
>  
> +<<<<<<< debian/changelog

I don't think this line and two others below should be here. They look like cruft from a merge conflict.

>    * Text-only fixes.
>  
>   -- Steve Langasek <steve.langasek at ubuntu.com>  Sat, 30 Sep 2023 22:24:30 -0700
> +=======
> +  * Drop various initramfs scripts that are no longer needed:
> +    - 07remove_oem_config: oem-config is only seeded in 'ship*' seeds, so
> +      not installed in any live systems; therefore it never needs to be
> +      removed.
> +    - 16disable_gpt_auto_mount: gpt generator no longer shipped in systemd.
> +    - 26serialtty: upstart-specific.
> +    - 32disable_hibernation: now disabled in policykit-desktop-privileges in
> +      desktop-common.
> +    - 36disable_trackerd: trackerd no longer exists.
> +    - 48kubuntu_disable_restart_notifications:
> +      /usr/share/kubuntu-default-settings/kde4-profile/ no longer exists,
> +      if this is still needed it needs to be implemented differently.
> +    - 49kubuntu_mobile_session: kubuntu-mobile-default-settings is obsolete.
> +    - 50ubiquity-bluetooth-agent: bluetooth-applet no longer exists.
> +    - 51unity8_wizard: unity8 no longer exists (ubuntu-unity is unity 7).
> +  * 12fstab: /etc/rcS.d/S*checkroot.sh is obsolete, no need to remove.
> +  * 15autologin: remove obsolete fix-up specific to kubuntu-active.
> +  * 15autologin: remove obsolete fix-up for
> +    /usr/share/xsessions/QLubuntu.desktop.
> +  * 18hostname: don't invoke hostname here, it will be set from the rootfs
> +    at boot anyway.
> +  * 25adduser: drop code for Ubuntu MID, which was obsolete before Lucid
> +    released in 2010.
> +  * 25configure_init: drop code specific to sysvinit, upstart, and
> +    ureadahead.
> +  * 31disable_update_notifier: drop handling of obsolete adept_notifier.
> +  * 34disable_kde_services: don't try to remove obsolete desktop files.
> +  * 35fix_language_selector: drop references to obsolete fontconfig-voodoo.
> +  * 35fix_language_selector:
> +    /usr/share/applications/language-selector.desktop no longer exists, but
> +    /usr/share/budgie-desktop/applications/language-selector.desktop does,
> +    so apply our fix there
> +
> + -- Steve Langasek <steve.langasek at ubuntu.com>  Sat, 30 Sep 2023 16:52:05 -0700
> +>>>>>>> debian/changelog
>  
>  casper (1.484) mantic; urgency=medium
>  
> diff --git a/scripts/casper-bottom/35fix_language_selector b/scripts/casper-bottom/35fix_language_selector
> index 96ea67c..cbad6a9 100755
> --- a/scripts/casper-bottom/35fix_language_selector
> +++ b/scripts/casper-bottom/35fix_language_selector
> @@ -20,15 +20,12 @@ esac
>  
>  log_begin_msg "$DESCRIPTION"
>  
> -if [ -e /rofs/usr/share/applications/language-selector.desktop ]; then
> +if [ -e /rofs/usr/share/budgie-desktop/applications/language-selector.desktop ]
> +then
>      sed '/^Exec/ s|/usr/bin/gnome-language-selector|& -n|' \
> -        /rofs/usr/share/applications/language-selector.desktop \
> -        > /root/usr/share/applications/language-selector.desktop
> +        /rofs/usr/share/budgie-desktop/applications/language-selector.desktop \
> +        > /root/usr/share/budgie-desktop/applications/language-selector.desktop

Does this even work? The idiom "sed ... file > file" should not be used. Maybe this works does to some quirks in ash, but I don't see how. The shell should truncate language-selector.desktop before sed even has a chance to open it. This should instead be "sed -i ... file".

>  
>  fi
>  
> -if [ -x /root/usr/bin/fontconfig-voodoo ]; then
> -    chroot /root fontconfig-voodoo --auto --quiet || true
> -fi
> -
>  log_end_msg


-- 
https://code.launchpad.net/~vorlon/casper/+git/casper/+merge/452496
Your team Unity7 Maintainers Team is requested to review the proposed merge of ~vorlon/casper:cleanup into casper:main.




More information about the Ubuntu-reviews mailing list