Page MenuHomeFreeBSD

Update graphics/showimage to 0.10 and take maintainership
ClosedPublic

Authored by tcberner on Mar 29 2017, 10:25 PM.
Tags
None
Referenced Files
Unknown Object (File)
Mon, Nov 25, 7:35 AM
Unknown Object (File)
Sat, Nov 23, 12:37 PM
Unknown Object (File)
Sat, Nov 23, 3:32 AM
Unknown Object (File)
Fri, Nov 22, 4:20 PM
Unknown Object (File)
Fri, Nov 22, 2:51 PM
Unknown Object (File)
Fri, Nov 22, 2:51 PM
Unknown Object (File)
Fri, Nov 22, 2:51 PM
Unknown Object (File)
Fri, Nov 22, 2:51 PM
Subscribers
None

Details

Summary
  • Update to 0.10 (kf5)
  • Rename the installed binary from showimage to showimage-kde (previously it was showimage-kde4, but that does not make much sense anymore)
  • Take maintainership.

Diff Detail

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

Event Timeline

graphics/showimage/files/patch-showimage.desktop
1 โ†—(On Diff #26795)

I don't see the point in this comment.
I get the idea, but this has the same problem as the trend of writing very bad commit messages explaining almost line by line what is being done in the commit.
The what is self-explanatory, yeah, the executable is renamed, I can see that.
A better comment would be "don't conflict with blah", and stop there, or, as it was already explained in another patch, just "see patch-foo".

Get rid of obvious comment.

graphics/showimage/Makefile
23 โ†—(On Diff #26808)

I don't really see the point in setting the id outside of the URL, it is not used anywhere else.

I also grep'ed the ports tree, and there does not seem to be any other use of that variable.

This revision is now accepted and ready to land.Mar 30 2017, 9:50 PM
This revision was automatically updated to reflect the committed changes.