Re: Refresh the xserver-xorg-video-openchrome package

[Emilio Pozuelo Monfort]

On 23/01/17 23:19, Dylan wrote:
> Hi,
> The package was broken due to bindnow hardening that prevent the
> loading of the driver [1].
> I have disabled it. Could someone upload the package to unstable?

Uploaded.

Cheers,
Emilio

Re: Refresh the xserver-xorg-video-openchrome package

[Dylan]

Hi,
The package was broken due to bindnow hardening that prevent the
loading of the driver [1].
I have disabled it. Could someone upload the package to unstable?

Thanks.

Best regards,
Dylan

[1] 
https://lists.freedesktop.org/archives/openchrome-users/2017-January/007309.html

Re: Refresh the xserver-xorg-video-openchrome package

[Dylan]

Hi Andreas,

2017-01-11 20:04 GMT+01:00 Andreas Boll :
> I noticed that this commit [1] missed an entry in the
> debian/patches/series file.
> Thus the patch "Reproducible_build.patch" won't be applied during the build.
> Could you add that missing entry?
> Otherwise the updated package looks good to me.

Thanks for pointing this mistake. :-)
It's corrected now.

Best regards,
Dylan

Re: Refresh the xserver-xorg-video-openchrome package

[Andreas Boll]

Hi Dylan,

I noticed that this commit [1] missed an entry in the
debian/patches/series file.
Thus the patch "Reproducible_build.patch" won't be applied during the build.
Could you add that missing entry?
Otherwise the updated package looks good to me.

Thanks,
Andreas

[1] 
https://anonscm.debian.org/cgit/pkg-xorg/driver/xserver-xorg-video-openchrome.git/commit/?id=e9f91e4b742035b3b9989f195497d0c27b38c291

2017-01-10 11:49 GMT+01:00 Dylan :
> Hi,
> Could someone upload the package to unstable?
> I have updated the package to fix the FTBFS on Hurd.
>
> Thanks.
>
> Best regards,
> Dylan
>

Re: Refresh the xserver-xorg-video-openchrome package

[Dylan]

Hi,
Could someone upload the package to unstable?
I have updated the package to fix the FTBFS on Hurd.

Thanks.

Best regards,
Dylan

Re: Refresh the xserver-xorg-video-openchrome package

[Dylan]

Hi Emilio,

2016-12-12 23:41 GMT+01:00 Emilio Pozuelo Monfort :
> Changes look good; uploaded.
>
> BTW please forward the patches upstream.

Thanks for the upload.

I will forward the patch that is not yet forwarded.

Best regards,
Dylan

Re: Refresh the xserver-xorg-video-openchrome package

[Emilio Pozuelo Monfort]

On 12/12/16 21:10, Dylan wrote:
> Hi
> I have pushed my changes on xxv-openchrome to git.debian.org.
> Could a DD check my changes and upload the package to unstable?

Changes look good; uploaded.

BTW please forward the patches upstream.

Cheers,
Emilio

Re: Refresh the xserver-xorg-video-openchrome package

[Dylan]

Hi
I have pushed my changes on xxv-openchrome to git.debian.org.
Could a DD check my changes and upload the package to unstable?

Thanks.

Best regards,
Dylan

Re: Refresh the xserver-xorg-video-openchrome package

[Dylan]

Hi,

2016-12-08 12:04 GMT+01:00 Andreas Boll :
>
> Welcome to the X Strike Force team!
>

:-)

> Please join pkg-xorg group on Alioth [1].

I'm waiting for the approbation to be able to push my changes.


Best regards,
Dylan

Re: Refresh the xserver-xorg-video-openchrome package

[Andreas Boll]

Hi Dylan,

Welcome to the X Strike Force team!

Please join pkg-xorg group on Alioth [1].
No need to push the package on mentors.
Just prepare it in git and some DD of the team will upload the package for you.
Feel free to join us on IRC [oftc] #debian-x

Thanks,
Andreas

[1] https://alioth.debian.org/projects/pkg-xorg/

2016-12-07 22:55 GMT+01:00 Dylan :
> Hi Andreas,
>
> 2016-12-05 14:31 GMT+01:00 Andreas Boll :
>> I have some comments for further improvements:
>>
>> Please use `wrap-and-sort -a -s` to be more consistent with the
>> previous style and other xsf packages.
>>
>> With the debhelper compat change you can also remove the build-dep on
>> dh-autoreconf since debhelper >= 10 depends on dh-autoreconf itself.
>> Furthermore you can remove autoreconf from the dh line in debian/rules
>> since debhelper compat level 10 enables it by default. See also [1].
>>
>> Other than that your changes look good to me.
>>
>
> Thanks for your review. I pushed your suggestions and some other changes.
>
> What do you prefer for the next steps (push on Alioth and the sponsorship)?
> Should I join the team to push myself these changes on the repository?
> Should I push the package on mentors.debian.net?
>
> Thanks
>
> Best regards,
> Dylan

Re: Refresh the xserver-xorg-video-openchrome package

[Dylan]

Hi Andreas,

2016-12-05 14:31 GMT+01:00 Andreas Boll :
> I have some comments for further improvements:
>
> Please use `wrap-and-sort -a -s` to be more consistent with the
> previous style and other xsf packages.
>
> With the debhelper compat change you can also remove the build-dep on
> dh-autoreconf since debhelper >= 10 depends on dh-autoreconf itself.
> Furthermore you can remove autoreconf from the dh line in debian/rules
> since debhelper compat level 10 enables it by default. See also [1].
>
> Other than that your changes look good to me.
>

Thanks for your review. I pushed your suggestions and some other changes.

What do you prefer for the next steps (push on Alioth and the sponsorship)?
Should I join the team to push myself these changes on the repository?
Should I push the package on mentors.debian.net?

Thanks

Best regards,
Dylan

Re: Refresh the xserver-xorg-video-openchrome package

[Andreas Boll]

Hi Dylan,

thanks much for working on xxv-openchrome!

I have some comments for further improvements:

Please use `wrap-and-sort -a -s` to be more consistent with the
previous style and other xsf packages.

With the debhelper compat change you can also remove the build-dep on
dh-autoreconf since debhelper >= 10 depends on dh-autoreconf itself.
Furthermore you can remove autoreconf from the dh line in debian/rules
since debhelper compat level 10 enables it by default. See also [1].

Other than that your changes look good to me.

Thanks,
Andreas

[1] 
https://anonscm.debian.org/cgit/pkg-xorg/lib/libdrm.git/commit/?id=d963fbc94795e4903f9ada320540380a3e774b7b

2016-12-04 0:31 GMT+01:00 Dylan :
> Hi the Debian X Strike Force team,
> The package of xserver-xorg-video-openchrome is quite outdated, with
> the reborn of openchrome project and the approaching of the Debian
> Freeze, it could be great to refresh the package.
>
> Since the package is RFA, I propose myself to update this package,
> maybe not to adopt in a first time. Let's see how I deal with it.
> Moreover, I don't have access to a supported hardware so a
> (co-?)maintainer with the hardware should be better.
>
> I am only DM, so I will need sponsor for this package. I have already
> started to update the package [1].
>
> Best regards,
> Dylan
>
> [1] https://bitbucket.org/Dybian/openchrome/
>