Ubuntu
xf86-video-msm package

  1. Bug #944709
  2. Comment #2

Comment 2 for bug 944709

Revision history for this message
Fabrice Coutadeur (fabricesp) wrote :

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