mysql-testsuite dependency change

Bug #1631338 reported by Lars Tangvald
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
mysql-5.7 (Ubuntu)
Fix Released
Low
Unassigned
Xenial
Fix Released
Low
Lars Tangvald

Bug Description

In MySQL 5.7.15 the only tests in the mtr suite written in python were rewritten in perl, to remove the python dependency, but this added a dependency on libjson-perl, without which those tests will fail.

Fix is to change the dependency in d/control

[Impact]
If libjson-perl isn't installed (which it won't be in an sbuild build) then the tests main.myisam_explain_json_non_select_all and main.myisam_explain_json_non_select_nonemysql-test-run will fail, both during the build and dep8 runs.

The change replaces the old python dependency with one on libjson-perl, and adds a build-dep on libjson-perl

[Test case]
Build package with sbuild and observe the test log

[Regression Potential]
This change has been in Yakkety for some time (5.7.15-0ubuntu2).

Removing the python dependency could cause issues if there were some unknown dependency on it still lingering.

Revision history for this message
Lars Tangvald (lars-tangvald) wrote :

Fix released in yakkety, in 5.7.15-0ubuntu2

Changed in mysql-5.7 (Ubuntu):
status: New → Fix Released
Mathew Hodson (mhodson)
tags: added: packaging
Changed in mysql-5.7 (Ubuntu):
importance: Undecided → Low
Changed in mysql-5.7 (Ubuntu Xenial):
importance: Undecided → Low
description: updated
Changed in mysql-5.7 (Ubuntu Xenial):
assignee: nobody → Lars Tangvald (lars-tangvald)
status: New → Fix Committed
Robie Basak (racb)
Changed in mysql-5.7 (Ubuntu Xenial):
milestone: none → xenial-updates
Revision history for this message
Andy Whitcroft (apw) wrote : Please test proposed package

Hello Lars, or anyone else affected,

Accepted mysql-5.7 into xenial-proposed. The package will build now and be available at https://launchpad.net/ubuntu/+source/mysql-5.7/5.7.17-0ubuntu0.16.04.2 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 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 to verification-done. If it does not fix the bug for you, please add a comment stating that, and change the tag to verification-failed. In either case, details of your testing will help us make a better decision.

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

tags: added: verification-needed
Revision history for this message
Lars Tangvald (lars-tangvald) wrote :

Verified that the dependency is correct in 5.7.17-0ubuntu0.16.04.2 and that the dep8 test suite passes

tags: added: verification-done
removed: verification-needed
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package mysql-5.7 - 5.7.17-0ubuntu0.16.04.2

---------------
mysql-5.7 (5.7.17-0ubuntu0.16.04.2) xenial; urgency=medium

  * Add libjson-perl dependency for test suite (LP: #1631338)
    The last two python tests in the mtr suite were rewritten
    in perl, and require this package. Added for mysql-testsuite
    and as a build-dep.
  * Build with DWITH_LZ4=system (LP: #1631339)
    Package was previously built with bundled liblz4, though it
    was specified as a build-dep.
  * Add support for custom datadir to systemd service (LP: #1574782)
    The service was reporting an error if no database could be found in
    /var/lib/mysql. It now checks the location specified in the config.
    Note that user must still handle apparmor access for custom datadir.
  * Fix copy of soft-link datadir to /var/lib/mysql-upgrade (LP: #1474212)
    Upgrade would sometimes fail if mysql-upgrade already contained a
    link copy from a previous upgrade.
  * Escape special characters in password (LP: #1598992)
    Special characters in the root password would cause syntax errors
    and postinst failures.
  * Failing tests on platforms supported by upstream fail build (LP: #1646488)
    A passing test suite is now enforced on i386 and amd64 platforms at
    build-time.
  * d/copyright: Updated with information about new source files
  * Unstable test main.xa_prepared_binlog_off disabled pending upstream fix.
    Upstream bug report: http://bugs.mysql.com/bug.php?id=83340
  * d/lintian-overrides: Updated line numbers

 -- Lars Tangvald <email address hidden> Fri, 03 Feb 2017 08:53:04 +0100

Changed in mysql-5.7 (Ubuntu Xenial):
status: Fix Committed → Fix Released
Revision history for this message
Brian Murray (brian-murray) wrote : Update Released

The verification of the Stable Release Update for mysql-5.7 has completed successfully and the package has now been released to -updates. Subsequently, the Ubuntu Stable Release Updates Team is being unsubscribed and will not receive messages about this bug report. In the event that you encounter a regression using the package from -updates please report a new bug using ubuntu-bug and tag the bug report regression-update so we can easily find any regressions.

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.