[request-tracker-maintainers] Help patching files in dependency package

[Ángel]

On 2023-01-17 at 13:38 +0100, Tobias Wackenhut wrote:
> Hi Andrew and Dominic,
> 
> thank you both for looking into this :) .
> 
> On 17.01.23 12:03, Andrew Ruthven wrote:
> > On Tue, 2023-01-17 at 10:10 +0000, Dominic Hargreaves wrote:
> > > However, Tobias, I don't see any patch that needs to be applied
> > > in
> > > <
> > > https://metacpan.org/release/BPS/RT-Extension-ResetPassword-1.12/source/README
> > > <
> > > https://metacpan.org/release/BPS/RT-Extension-ResetPassword-1.12/source/README
> > > >>
> > > - Debian already has RT 5.0.3 (>= 5.0.2)?
> > 
> > Support for RT5 was introduced in RT::Extension::ResetPassword
> > v1.07 
> > which is in Sid & Bookworm.
> 
> Hmm, I'm talking about the install section of the README where it
> says:
> 
> [...]
>      Install Patches
>          This is to enable searches for users with a password set.
> 
>          For RT 5 prior to 5.0.2, apply this:
> 
>              patch -p1 -d /opt/rt5 < patches/user-admin-
> callbacks.patch
> [patch files for other versions]

The key is "For RT 5 prior to 5.0.2"
The package request-tracker5 is at version 5.0.3+dfsg-1.1, so those
instructions can be skipped.


Andrew: I would change the Depends for request-tracker5 on rt5-
extension-resetpassword to request-tracker5 >= 5.0.2 
 

> [...]
> 
> > I have updated packages for 1.12, they just haven't been uploaded
> > yet. 
> > I've just pushed the branches to Salsa.
> 
> Uuh, cool. I was looking at what changes have been made and so far
> could not find anything related to these patches. Are these still
> necessary?

That's not necessary for rt5-extension-resetpassword

rt4-extension-resetpassword *would* require the core RT patches (to be
included in request-tracker4) But I'm not convinced it's worth the
effort (how many people use this extension on rt4?), it might be easier
to drop rt4-extension-resetpassword

Regards


>