ftfbs on armhf
Bug #944709 reported by
Marcin Juszkiewicz
This bug affects 2 people
| Affects | Status | Importance | Assigned to | Milestone | |
|---|---|---|---|---|---|
| xf86-video-msm (Debian) |
Fix Released
|
Unknown
|
|||
| xf86-video-msm (Ubuntu) |
Fix Released
|
Undecided
|
Unassigned | ||
Bug Description
Package ftfbs on armhf due to use of wrong compiler flags.
Related branches
Revision history for this message
| Marcin Juszkiewicz (hrw) wrote | #1 |
| Changed in xf86-video-msm (Ubuntu): | |
| status: | New → Triaged |
Revision history for this message
| Fabrice Coutadeur (fabricesp) wrote | #2 |
| Changed in xf86-video-msm (Ubuntu): | |
| status: | Triaged → Incomplete |
Revision history for this message
| Marcin Juszkiewicz (hrw) wrote | #3 |
| Changed in xf86-video-msm (Ubuntu): | |
| status: | Incomplete → Opinion |
Revision history for this message
| Fabrice Coutadeur (fabricesp) wrote | #4 |
| Changed in xf86-video-msm (Ubuntu): | |
| status: | Opinion → Fix Committed |
Revision history for this message
| Launchpad Janitor (janitor) wrote | #5 |
| Changed in xf86-video-msm (Ubuntu): | |
| status: | Fix Committed → Fix Released |
| Changed in xf86-video-msm (Debian): | |
| status: | Unknown → New |
| Changed in xf86-video-msm (Debian): | |
| status: | New → Fix Released |
To post a comment you must log in.
Hi Marcin,
I've seen some additional changes in the patch sent to Debian, especially the part changing the control file, to add armhf to the packages. Could you please do the following changes to your defdiff:
- add a DEP3 header to the patch (giving credit to the creator of this patch)
- Add justification why you are dropping an upstream cflag (the explanation given in the debian bug seems fine) in the patch header
- change debian/control so that the packages can be installed on armhf.
generally speaking, it's worth checking Debian PTS before submitting a patch to Ubuntu, to check if a solution has already been given or not.
Anyway, thanks a lot for your work!
Fabrice