Page MenuHomeFreeBSD

net/remmina and others: remmina port update and cleanup
ClosedPublic

Authored by darkfiberiru_gmail.com on Tue, Aug 6, 5:01 AM.

Details

Summary

Update to bring Remmina to 1.3.5 and cleanup of some minor port details.

Adding support for ST("simple" terminal), www, and spice. Enabling nx by default.

Telepathy plugin is disabled upstream and is removed as a port to match.

Test Plan

Testing ports build are clean with portlint and poudriere testport and that I can use the plugins I have easy access to (rdp/vnc/exec/st/rdp)

Any other testing anyone else can do is much appreciated

Diff Detail

Repository
rP FreeBSD ports repository
Lint
Lint Skipped
Unit
Unit Tests Skipped

Event Timeline

Missed a telepathy reference for it's description. (Purposely left references in the plugin mk file in case telepathy is reactivated in future)

linimon retitled this revision from Remmina Port update and cleanup to net/remmina and others: remmina port update and cleanup.Tue, Aug 6, 3:10 PM
woodsb02 accepted this revision.Wed, Aug 7, 10:56 PM

Hi Nick,
Thanks for submitting this diff - it looks like great work, and I am happy for it to be committed.
Given you are not using an @freebsd.org account here, I assume you do not have a commit bit? Would you like me to commit this on your behalf?
Regards,
Ben

This revision is now accepted and ready to land.Wed, Aug 7, 10:56 PM

Thanks for the greenlight if you can commit it that would be great.

This only minor comment I would make, is that I prefer to keep things alphabetical in Makefile entries. I have pointed these out with inline comments, but happy for this to be committed without these being changed, as it is only a preference.

net/remmina-plugins/bsd.plugin.mk
73

Add in alphabatical order (w before x)

76

Add in alphabetical order (k before w)

net/remmina/Makefile
16

Add in alphabetical order (sodium before soup)

18

Ensure alphabetical order

woodsb02 added inline comments.Wed, Aug 7, 11:03 PM
net/remmina-plugin-vnc/Makefile
12

Ensure alphabetical

Of course I will get those cleaned up slipped through my final checks.

One more comment on how to submit phabricator reviews - it is best if the full context of the diff is submitted (allowing me to expand the other parts of the files in between the changes if I am looking for more context).
This can be done using the "arc" command, or by generating the diff with "svn diff --diff-cmd=diff -x -U999999 > change.diff".
This is described here: https://wiki.freebsd.org/Phabricator

Covered all wanted alphabetical fixes plus a few more I found during review

This revision now requires review to proceed.Sun, Aug 11, 12:57 AM
mat added inline comments.Tue, Aug 13, 2:06 PM
net/remmina-plugin-exec/Makefile
8–9

Those variables are not defined before, no need for +=. (same for all slave ports later.)

net/remmina/pkg-message
0–9

Please convert this to ucl so that it is only shown on install.

darkfiberiru_gmail.com marked 2 inline comments as done.Tue, Aug 13, 7:54 PM

Just realized that I missed the added ports(remmina plugins) will also need to add those.

Fixed missing plugins/ports and issues raised above.

darkfiberiru_gmail.com marked 4 inline comments as done.Tue, Aug 13, 8:18 PM

Updated diff to fix issues.

net/remmina/pkg-message
0–9

Does this look good now?

darkfiberiru_gmail.com marked an inline comment as done.Tue, Aug 13, 8:19 PM

@woodsb02 I think your clear to commit whenever you have time.

woodsb02 accepted this revision.Sun, Aug 18, 2:02 PM

This has now been committed, and this review can be closed (not sure why this wasn't automatically picked up by phabricator):
https://reviews.freebsd.org/rP509197

Thanks again for a quality submission.

net/remmina/Makefile
16

I had to move the " \" to the second last line when I committed this

This revision is now accepted and ready to land.Sun, Aug 18, 2:02 PM

Closed by commit r509197