Page MenuHomeFreeBSD

devel/geany: Add option to install geany-themes (invert dependency from geany-themes)
ClosedPublic

Authored by james.wright_digital-chaos.com on Nov 22 2019, 7:59 PM.
Referenced Files
Unknown Object (File)
Sat, Mar 16, 3:38 AM
Unknown Object (File)
Sat, Mar 16, 3:38 AM
Unknown Object (File)
Sat, Mar 16, 3:37 AM
Unknown Object (File)
Sat, Mar 16, 3:25 AM
Unknown Object (File)
Tue, Mar 12, 7:56 AM
Unknown Object (File)
Tue, Mar 12, 7:56 AM
Unknown Object (File)
Jan 14 2024, 4:19 PM
Unknown Object (File)
Jan 12 2024, 4:37 PM
Subscribers

Details

Summary

devel/geany: Added new option to install additional themes (devel/geany-themes).
devel/geany-themes: Removed dependency on "geany" to avoid circular dependency, bumped PORTREVISION.

Diff Detail

Repository
rP FreeBSD ports repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

Changed devel/geany/Makefile to remove THEMES option from default options.

Thanks for the patch. I'll commit it later, only changing the RUN_DEPENDS line.

devel/geany/Makefile
49 ↗(On Diff #64752)

This is wrong in this case. The geany port installs a file in this directory itself (alt.conf, a color theme included in geany distribution), so the directory will be there if the port is already installed.

I'll change this to

THEMES_RUN_DEPENDS=${LOCALBASE}/share/geany/colorschemes/bespin.conf:devel/geany-themes

referencing a specific file installed by geany-themes.

This revision is now accepted and ready to land.Nov 23 2019, 1:32 PM

I forgot, there is also no need to bump geany PORTREVISION, since you're only adding a non default option. No need to force rebuilding the package/reinstalling.

It would have been necessary if you had made the new option the default.

Removed PORTREVISION bump from devel/geany/Makefile, and improved dependency check to test for a spefic colour theme file as suggested.

This revision now requires review to proceed.Nov 23 2019, 3:02 PM
This revision is now accepted and ready to land.Nov 23 2019, 3:32 PM