Page MenuHomeFreeBSD

Clarify the INSTALLS_ICONS text a bit.
ClosedPublic

Authored by kwm on Jun 26 2015, 9:22 AM.
Tags
None
Referenced Files
Unknown Object (File)
Tue, Nov 26, 8:30 AM
Unknown Object (File)
Sun, Nov 24, 6:01 PM
Unknown Object (File)
Nov 14 2024, 2:27 PM
Unknown Object (File)
Nov 7 2024, 9:31 AM
Unknown Object (File)
Nov 1 2024, 3:23 PM
Unknown Object (File)
Oct 28 2024, 3:29 PM
Unknown Object (File)
Oct 18 2024, 6:32 AM
Unknown Object (File)
Oct 18 2024, 3:37 AM
Subscribers

Details

Summary

Clarify the text to point out that it only used for GTK applications, and
so that it is not needed for QT applications.

Diff Detail

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

Event Timeline

kwm retitled this revision from to Clarify the INSTALLS_ICONS text a bit..
kwm updated this object.
kwm added reviewers: wblock, rene.
antoine added a reviewer: antoine.
antoine added a subscriber: antoine.

Macro lgtm:

This revision is now accepted and ready to land.Jun 26 2015, 9:28 AM

Some picky notes that can mostly be ignored. But I am curious about the icon caching. Thanks!

Mk/bsd.gnome.mk
64 ↗(On Diff #6466)

In doc, we try to avoid "you" and "your" and would use something more like "If a GTK+ port..., it should use this macro." Comments in Makefiles are not subject to the same rules, but that still sounds better to me.

66 ↗(On Diff #6466)

"If the icons are not cached, they will not be displayed." Sorry, does this mean using this macro will cache the icons? So is it really saying "Using this macro insures that icons are cached and will display correctly."?

67 ↗(On Diff #6466)

Our doc guidelines suggest spelling out contractions. (Yes, I know, this is a comment. But it's a kind of doc.)

rene edited edge metadata.

With Warren's comments I guess, but maybe there are more places in the Ports Tree which could use English cleanup...

kwm edited edge metadata.

Tweaked the text after warren's comments.

This revision now requires review to proceed.Jun 29 2015, 9:04 AM
bapt added a reviewer: bapt.
This revision is now accepted and ready to land.Jun 29 2015, 9:06 AM
This revision was automatically updated to reflect the committed changes.