[DRE-maint] Bug#1002532: pygments breaks ruby-pygments.rb autopkgtest: UTF-8 != ASCII-8BIT
Mattia Rizzolo
mattia at debian.org
Thu Jan 20 17:50:33 GMT 2022
Hi Alex,
On Thu, Jan 06, 2022 at 11:19:00AM +0100, Alexandre Ghiti wrote:
> As the current version we have is from 2017, I bumped the version of
> this package to the latest available version: I updated the patches,
> removed the ones that do not apply anymore, updated the build system
> and dependencies. The result is available in my PPA [2] and fixes the
> issue we encounter here.
>
> Can you consider pulling this?
Thank you for this!!
I had a look at your work, however I couldn't help but notice that:
* the .orig you used looks odd, much larger than what I get from uscan
(despite yours is also using a different compression, so repacked).
* You did a ton of patch wrangling, including deletion, renaming,
rebasing, etc. which is all fine, except that the way you did it
obscures quite a bit what you did. Why did you drop the numbers from
the patches? Do you have --no-patch-numbers as you gbp-pq default or
something?
* why requiring gem2deb >=1 ? that's already in bullseye as well in
focal, so why did you feel the need to add the version? (that's also
not in the changelog)
As such, I went ahead and re-imported the repacked origin I got myself
in git.
Could I ask you to submit a MR on top of it, with at least commits
separating the deletion, refresh and rebasing of patches (and eventual
new ones, I can't tell at a glance if any new patch appeared…) also
separated.
https://salsa.debian.org/ruby-team/ruby-pygments.rb
Thank you for your work!! :)
--
regards,
Mattia Rizzolo
GPG Key: 66AE 2B4A FCCF 3F52 DA18 4D18 4B04 3FCD B944 4540 .''`.
More about me: https://mapreri.org : :' :
Launchpad user: https://launchpad.net/~mapreri `. `'`
Debian QA page: https://qa.debian.org/developer.php?login=mattia `-
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 833 bytes
Desc: not available
URL: <http://alioth-lists.debian.net/pipermail/pkg-ruby-extras-maintainers/attachments/20220120/20ab1c60/attachment-0001.sig>
More information about the Pkg-ruby-extras-maintainers
mailing list