search

OpenPrinting / libcupsfilters

Apache License 2.0
5 stars 22 forks source link

Remove unused dependencies #7

Closed zdohnal closed 1 year ago

zdohnal commented 1 year ago

Avahi, Glib, Freetype and Zlib are used in other projects or removed as whole.

tillkamppeter commented 1 year ago

Thanks a lot, these dependencies got so many with the years, it is easy to overlook some of them ...

zdohnal commented 1 year ago

Np! :) I write comments for BuildRequires in my spec file, so I can easier track what is no longer needed - maybe do a bullet list in INSTALL file for dependencies with explanation why they are needed?

It won't prevent the situation when you remove a feature and don't read the INSTALL, but at least maintainers can sometimes check the deps and when they see the dep in INSTALL and git says the feature which needed the dependency is gone, they can report it.

tillkamppeter commented 1 year ago

Could you add these comments which you put into BuildRequires also to an appropriate place in the source code documentations of the OpenPrinting repos, into README.md or INSTALL or so? Thanks in advance.