[Pkg-roundcube-maintainers] Cleanup roundcube

Sandro Knauß hefee at debian.org
Wed Jan 27 02:42:48 GMT 2021


Hey,

after some month I found time again to look at the roundcube package and I 
have no glue at all what have changed. It seems to me that this can have 
another cleanup, so I'll through some thoughts in the room, maybe some are 
non-sense, as I do not know every reason, why you did your work like you it. 
And I'm happy that you took over! So this is not to criticize your work, I 
just want to help. 

* I not found time to removed the patch that failed for others: retry-to-
reach-imap-server.patch it would be nice if we get this into next stable. 
Should I do a release or are there any planed changes from your side in the 
pipeline?

I created now a plugin for this in upstream to have the same functionality 
without the negative effects: 
https://github.com/roundcube/roundcubemail/pull/7844

* copy of install-jsdeps.sh inside debian folder. As I marked in the header 
once, it can be replaced with the version shipped from roundcube > 1.3.0. But 
I think it is not used anymore anyways. So we can remove it?

* the d/rules looks quite complicated aren't there things that should better 
been part of debhelper (some buildsystem etc.) I normally do KDE packages 
these days and there the rules file just have two or three lines ;) But it is 
not obvious what different parts can be bundled, so other packages also can 
profit from this work.

* a lot of d/rules section can be moved to execute_after_dh_XXX (when it is 
fine to use debhelper 13 also in backports)

Many many thanks for all your work you do for the roundcube package!

Regards

hefee


-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 833 bytes
Desc: This is a digitally signed message part.
URL: <http://alioth-lists.debian.net/pipermail/pkg-roundcube-maintainers/attachments/20210127/e40a2f79/attachment.sig>


More information about the Pkg-roundcube-maintainers mailing list