Differential D33
Clean up Calamares settings. Authored by kc2bez on Aug 18 2019, 11:37 PM.
Details
This resolves T107 Refactor automirror Add donate button url and geoip to welcome Adjust locale.conf geoip format Set default filesystem type and no user swap choices Add defaults to settings to eliminate warnings Add efi mount options to fstab.conf Set restart mode in finished.conf to new style Set bootloader entry name to Ubuntu and slideshowAPI to 1 Test Calamares installation for proper location detection. Test Calamares installation for proper efi mount. Test Calamares installation for proper bootloader entry.
Diff Detail
Event TimelineComment Actions LGTM. I didn't pine over every single key, but I assume you have. Land, upload, and tag away. We'll shake it out when we've got a new ISO. Comment Actions After sbuild -d eoan-amd: N: Processing source package calamares-settings-ubuntu (version 1:19.10.4, arch source) ... W: calamares-settings-ubuntu source: debian-rules-sets-dpkg-architecture-variable DEB_HOST_MULTIARCH (line 3) I: calamares-settings-ubuntu source: testsuite-autopkgtest-missing E: calamares-settings-ubuntu source: not-binnmuable-all-depends-any calamares-settings-lubuntu -> calamares-settings-ubuntu-common N: ---- N: Processing buildinfo package calamares-settings-ubuntu (version 1:19.10.4, arch all amd64) ... N: ---- N: Processing binary package calamares-settings-ubuntu-common (version 1:19.10.4, arch amd64) ... I: calamares-settings-ubuntu-common: extended-description-is-probably-too-short E: calamares-settings-ubuntu-common: python-script-but-no-python-dep usr/lib/x86_64-linux-gnu/calamares/modules/automirror/main.py #!python3 W: calamares-settings-ubuntu-common: executable-not-elf-or-script usr/lib/x86_64-linux-gnu/calamares/modules/automirror/automirror.conf W: calamares-settings-ubuntu-common: executable-not-elf-or-script usr/lib/x86_64-linux-gnu/calamares/modules/automirror/module.desc N: ---- N: Processing binary package calamares-settings-lubuntu (version 1:19.10.4, arch all) ... I: calamares-settings-lubuntu: extended-description-is-probably-too-short W: calamares-settings-lubuntu: uses-dpkg-database-directly usr/bin/calamares-logs-helper W: calamares-settings-lubuntu: binary-without-manpage usr/bin/calamares-logs-helper I: calamares-settings-lubuntu: desktop-entry-lacks-keywords-entry usr/share/applications/lubuntu-calamares.desktop W: calamares-settings-lubuntu: desktop-command-not-in-package usr/share/applications/lubuntu-calamares.desktop bash W: calamares-settings-lubuntu: script-not-executable usr/share/applications/lubuntu-calamares.desktop W: calamares-settings-lubuntu: unusual-interpreter usr/share/applications/lubuntu-calamares.desktop #!xdg-open N: Finished processing group calamares-settings-ubuntu/1:19.10.4 Not sure, which error can be removed easily.
Comment Actions
Comment Actions Please drop the shebang on the desktop and then you won't need to override unusual-interpreter or script-not-executable though I still advise removing chmod +x lubuntu/*.desktop from the root Makefile. Also it looks like it's executable. You can undo that. Your issue with executables in automirror (executable-not-elf-or-script) is that debian/rules sets chmod -R 755 debian/calamares-settings-ubuntu-common/usr/lib/. That should probably be rewritten to chmod -R 755 debian/calamares-settings-ubuntu-common/usr/lib/${DEB_HOST_MULTIARCH}/calamares/modules/automirror/main.py because at least that should be executable. You got a calamares misspell in your changelog, too. Finally, let's us all never forget than anything to do with usr/bin/calamares-logs-helper is entirely the fault of @tsimonq2. Comment Actions
| ||||||||||||||||||||||