autopkgtest regression TEST-22-TMPFILES are not executable

Bug #1804864 reported by Dimitri John Ledkov on 2018-11-23
10
This bug affects 1 person
Affects Status Importance Assigned to Milestone
systemd (Ubuntu)
Undecided
Unassigned
Bionic
Undecided
Unassigned
Cosmic
Undecided
Unassigned

Bug Description

[Impact]

 * Newly added testcase in a security upload used a traditional GNU patch format, instead of using extended git patch format, as supported by GNU patch. Therefore, executable bits on shells scripts were lost, resulting in autopkgtest failures.

[Test Case]

 * `upstream` test/TEST-22-TMPFILES autopkgtest case should pass.

[Regression Potential]

 * This is testcode change only.

FYI for others checking for current known test issues like me - this is in systemd/239-7ubuntu12 currently in -proposed

Due to that I'd think this must at least be triaged as there is this changelog entry:
  * test: Set executable bits on TEST-22-TMPFILES shell scripts. (LP: #1804864)
    File: debian/patches/test-Set-executable-bits-on-TEST-22-TMPFILES-shell-script.patch
    https://git.launchpad.net/~ubuntu-core-dev/ubuntu/+source/systemd/commit/?id=0e5b6e44a962f299565949e1006a4ba86d171dc3

Changed in systemd (Ubuntu):
status: New → Triaged
summary: - autopkgtest regression TEST-22-TMPFILES are note executable
+ autopkgtest regression TEST-22-TMPFILES are not executable
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package systemd - 239-7ubuntu14

---------------
systemd (239-7ubuntu14) disco; urgency=medium

  * Fix compat with new meson.
    File: debian/patches/meson-rename-Ddebug-to-Ddebug-extra.patch
    https://git.launchpad.net/~ubuntu-core-dev/ubuntu/+source/systemd/commit/?id=3b764ec1b76768a8c40635019fa5a8acb81b223e

 -- Dimitri John Ledkov <email address hidden> Thu, 29 Nov 2018 16:53:00 +0000

Changed in systemd (Ubuntu):
status: Triaged → Fix Released
Changed in systemd (Ubuntu Cosmic):
status: New → In Progress

Hello Dimitri, or anyone else affected,

Accepted systemd into cosmic-proposed. The package will build now and be available at https://launchpad.net/ubuntu/+source/systemd/239-7ubuntu10.5 in a few hours, and then in the -proposed repository.

Please help us by testing this new package. See https://wiki.ubuntu.com/Testing/EnableProposed for documentation on how to enable and use -proposed. Your feedback will aid us getting this update out to other Ubuntu users.

If this package fixes the bug for you, please add a comment to this bug, mentioning the version of the package you tested and change the tag from verification-needed-cosmic to verification-done-cosmic. If it does not fix the bug for you, please add a comment stating that, and change the tag to verification-failed-cosmic. In either case, without details of your testing we will not be able to proceed.

Further information regarding the verification process can be found at https://wiki.ubuntu.com/QATeam/PerformingSRUVerification . Thank you in advance for helping!

N.B. The updated package will be released to -updates after the bug(s) fixed by this package have been verified and the package has been in -proposed for a minimum of 7 days.

Changed in systemd (Ubuntu Cosmic):
status: In Progress → Fix Committed
tags: added: verification-needed verification-needed-cosmic
Dimitri John Ledkov (xnox) wrote :

========== TEST-22-TMPFILES ==========
make: Entering directory '/tmp/autopkgtest.K5eYsC/build.9vm/src/test/TEST-22-TMPFILES'
TEST CLEANUP: Tmpfiles related tests
TEST SETUP: Tmpfiles related tests
...
TEST RUN: Tmpfiles related tests
...
TEST RUN: Tmpfiles related tests [OK]
make: Leaving directory '/tmp/autopkgtest.K5eYsC/build.9vm/src/test/TEST-22-TMPFILES'

autopkgtest [02:14:17]: test upstream: -----------------------]
upstream PASS

the test case succeeds, and thus has the right executable permission.

tags: added: verification-done verification-done-cosmic
removed: verification-needed verification-needed-cosmic
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers