Code review comment for ~bryce/ubuntu/+source/clamav:merge-v0.103.7adfsg-1-lunar

Revision history for this message
Bryce Harrington (bryce) wrote :

> Did you find where they came from?
> That upload was yours or am I completely mistaken?

Yes, that was from my upload. It's not in my git branch though, thus my surprise. I did a bit more digging to understand it.

It looks like clamav is set up to do this automatically during `debuild -S`:

    # d/rules
    override_dh_auto_clean:
        # Update .po files for debconf.
 debconf-updatepo
        dh_auto_clean

So it seems to be an intentional behavior, although I can't recall running across this with other packages. I'm curious why there are changes needed to the po files; the delta we carry does not affect strings at all, so this seems due just to some O/S skew between us and Debian.

If this were an SRU I'd be a bit more concerned about including such changes, but since this is a merge it seems just part of the process I guess. I don't see a documented way to skip it other than commenting out the debconf-updatepo line above.

Anyway, other than the surprise in my git branch, this behavior seems benign so I'm leaving it as is.

« Back to merge proposal