Please run the test suite on build by default

Bug #451998 reported by Martin Pitt
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
m2crypto (Ubuntu)
Fix Released
Medium
Kees Cook
Karmic
Fix Released
Medium
Kees Cook

Bug Description

Binary package hint: m2crypto

This will catch regressions early (also architecture specific ones) and avoid introducing nasty bugs, which is especially important for security related software like this.

(Split off from MIR bug 434723)

Revision history for this message
Martin Pitt (pitti) wrote :
Changed in m2crypto (Ubuntu):
assignee: nobody → Zulnedi (zul)
importance: Undecided → Medium
status: New → Triaged
assignee: Zulnedi (zul) → Chuck Short (zulcss)
Revision history for this message
Kees Cook (kees) wrote :

This issue appears to be a problem with the builders -- they are setting HUP to "ignore", which is what the testsuite uses to shut down the spawned SSL servers. I've switched the signal to 9 instead, and the testsuite passes fine on the builders.

Changed in m2crypto (Ubuntu):
assignee: Chuck Short (zulcss) → Kees Cook (kees)
status: Triaged → Fix Committed
Changed in m2crypto (Ubuntu Karmic):
milestone: none → ubuntu-9.10
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package m2crypto - 0.19.1-0ubuntu7

---------------
m2crypto (0.19.1-0ubuntu7) karmic; urgency=low

  * debian/rules: enable testsuite, add more files to "clean" rule.
  * tests/test_ssl.py: use signal 9 to kill old s_server processes
    to work around buildd HUP signal-ignore-mask (LP: #451998).

 -- Kees Cook <email address hidden> Fri, 16 Oct 2009 12:51:15 -0700

Changed in m2crypto (Ubuntu Karmic):
status: Fix Committed → Fix Released
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.