Code review comment for lp://qastaging/~roadmr/ubuntu/oneiric/checkbox/0.12.7

Revision history for this message
Daniel Manrique (roadmr) wrote :

> Is there a reason Architecture: in debian/control is changed from "all" to
> "any"? I don't see any architecture-dependent code being produced now.

Hi Daniel,

Yes, please see bug 833696 for the specific reason to do architecture-specific builds. Checkbox builds two binaries, threaded_memtest and clocktest, and as the bug mentions, they wouldn't run on amd64 systems, and this was the best solution to that problem.

This affects only the checkbox base package which contains the core code (all in Python, not arch-specific) and test definitions (most are just python or shell scripts, save for the two I mention earlier). All the additional packages (including the GUI, checkbox-gtk) are still Architecture: all.

Thanks!

« Back to merge proposal