[noble/shotwell] Package Phasing Stopped

[Steve Langasek]

On Thu, Oct 17, 2024 at 10:29:44AM +0200, Julian Andres Klode wrote:
> On Wed, Oct 16, 2024 at 03:47:42PM GMT, Steve Langasek wrote:
> > Hi Nathan,

> > On Mon, Oct 14, 2024 at 10:30:26AM +0200, Nathan Teodosio wrote:
> > > Hi, this error is no regression as per the table of the bug:

> > > 23.04   23.10   24.04
> > > 336	234	5

> > > So if nobody disagrees I'll file a merge request to except it.

> > This shows that the error in question was seen in both 23.04 and 23.10. 
> > However, looking at the error page, it appears that 0.32.6-1ubuntu2 is the
> > first version of this package where the error was reported *in noble*. 
> > Since this is an SRU, obviously this is not the first version of the package
> > in the noble release; and what we care about in an SRU is avoiding
> > regressions for users who have already installed that stable release.

> > So no, the above table does not demonstrate that it's not a regression, and
> > I would nack a merge request for this with only the above as rationale.

> This all seems sort of expected looking at the commits. The original
> commit fixed the crash but introduced the memory leak. So it was
> reverted for causing a regression (and not believing it fixed a crash),
> but of course now the crash is back.

Thanks for this analysis.

Based on this, I'm divided on the question of whether we should override the
phasing; which is worse, the memory leak and an eventual OOM crash, or a
crash that happens right away?

But https://bugs.launchpad.net/ubuntu/+source/shotwell/+bug/2084777 was
opened for this particular crash and has already been uploaded to plucky,
with intent that this also be SRUed to noble and oracular; so maybe we
should just push ahead with that and side step having to decide between two
bugs.

-- 
Steve Langasek                   Give me a lever long enough and a Free OS
Debian Developer                   to set it on, and I can move the world.
Ubuntu Developer                                   https://www.debian.org/
slangasek at ubuntu.com                                     vorlon at debian.org
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 833 bytes
Desc: not available
URL: <https://lists.ubuntu.com/archives/ubuntu-release/attachments/20241022/c839d333/attachment.sig>