Hi,

Regarding scripts in tools/dist:

backport.pl is a heavy user of "given ... when ..." and given that these
constructs will be removed[1] when Perl 5.42 is released (pun intended), we
need to take some action. I'm guessing we have about one year before
hitting a brick wall.

backport.pl is "considered deprecated" according README.backport, but it
seems danielsh added this on his own - I can't find a discussion or
decision on dev@.

Both scripts can handle the nightly backport job (running on svn-qavm) as
well as the backport conflict detection (currently on ci2.apache.org).
backport.pl also has an interactive mode where it can review existing
nominations and create new ones in the correct format.

I see two options:
* Rewrite backport.pl to use if/else blocks instead. This should be quite
easy but I don't think we have a test suite to verify the result so we
might find regressions further down the line.
* Bite the bullet, delete backport.pl and officially switch to backport.py.

The first option seems like the simplest here-and-now, but we are still in
the position where we have two different scripts doing more or less the
same.

For the second option, we would loose the interactive features but I
believe we have more developers fluent in Python so we can better support
it going forward. If we select this option, we either need to decide that
the interactive features can go away, or we need to spend the resources
rebuilding them in Python. We also need to do changes on svn-qavm, I can
volunteer to do this.

I'm leaning towards the second option. The interactive features are nice
but I usually don't use them so I wouldn't mind loosing them - I think the
advantage of a more widely used language is more important.

Thought and comments?

Kind regards,
Daniel


[1] https://perldoc.perl.org/perldeprecation#Perl-5.42

Reply via email to