[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Re: Updating mupen64plus-qt



Hi Dan,

On Tue, Feb 18, 2020, 12:33 Dan <maintainer@hasting.email> wrote:
On 2/12/20 9:00 PM, Dan wrote:
> On 2/12/20 5:40 PM, Markus Koschany wrote:
>> Hi Dan,
>>
>> Am 12.02.20 um 04:32 schrieb Dan:
>> [...]
>>> Hi Markus,
>>>
>>> There's a new version of this package I've uploaded to
>>> https://salsa.debian.org/games-team/mupen64plus-qt/
>>>
>>> I've written into pkg-games-devel@lists.alioth.debian.org a couple times
>>> the past few weeks with no responses. Wondering if you could get this
>>> reviewed/uploaded for me or point me to a good channel to contact for
>>> getting this updated.
>>>
>>> Thanks!
>>> - Dan
>>
>> Thanks, I've uploaded your new upstream version to unstable. You forgot
>> to import the new version to our upstream and pristine-tar branches
>> though, so the checksum for the orig tarball will be different from the
>> official release. Otherwise the changes looked good to me.
>>
>> Cheers,
>>
>> Markus
>>
>
> Markus,
>
> Thanks for uploading! Thought I had imported those branches with gbp but perhaps I forgot to run `git push --tags`
>
> - Dan

Hi Markus,


I've noticed the new package produces an icon error:
https://appstream.debian.org/sid/main/issues/mupen64plus-qt.html

On my system it's placing the icon in /usr/share/icons/hicolor/128x128/apps/mupen64plus-qt.png/mupen64plus.png (inside a directory)

Seems that's caused by this commit that was commited right before it was uploaded. It removed the cp command that prevented this issue:
https://salsa.debian.org/games-team/mupen64plus-qt/commit/7d9c402244aeb62fe634f04e8d66543021778bc3

What's the best way to proceed? I can revert the commit and upload a new version if needed.

I would not revert that commit, it is *almost* perfect as-is. Just delete the "mupen64plus-qt.png" at the end of line 3 in d/install.

-Olek

Reply to: