Bug 198492 - net/remmina and net/remmina-plugins: Update to 1.2
Summary: net/remmina and net/remmina-plugins: Update to 1.2
Status: Closed Feedback Timeout
Alias: None
Product: Ports & Packages
Classification: Unclassified
Component: Individual Port(s) (show other bugs)
Version: Latest
Hardware: Any Any
: --- Affects Many People
Assignee: freebsd-ports-bugs (Nobody)
URL:
Keywords: needs-patch, needs-qa
Depends on:
Blocks:
 
Reported: 2015-03-10 12:48 UTC by muxas
Modified: 2016-05-23 01:16 UTC (History)
2 users (show)

See Also:


Attachments
The archive contains patches for 2 ports: net/remmina and net/remmina-plugins (222.54 KB, application/gzip)
2015-03-10 12:48 UTC, muxas
no flags Details
net/remmina-plugins patch (2.47 KB, patch)
2015-12-09 12:17 UTC, Pavel Timofeev
no flags Details | Diff
net/remmina-plugin-rdp patch (507 bytes, patch)
2015-12-09 12:18 UTC, Pavel Timofeev
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description muxas 2015-03-10 12:48:23 UTC
Created attachment 154138 [details]
The archive contains patches for 2 ports: net/remmina and net/remmina-plugins

Hi!

I upgraded remmina ports to the latest stable version 1.1.2. Also I changed the default gnome environment that the ports are compiled for from 2 to 3.

Ports changed:
net/remmina
net/remmina-plugins

Files added/removed for net/remmina:
> svn add files/patch-remmina-plugins_CMakeLists.txt.diff
> svn add files/patch-remmina-plugins_vnc_CMakeLists.txt.diff
> svn add files/patch-remmina_CMakeLists.txt.diff

No files were added or removed for net/remmina-plugins.

These ports are dependant so I put them into one report. Please, see attached archive with patches.

Best regards,
Maxim Vetrov
Comment 1 muxas 2015-03-11 02:20:12 UTC
Hi,

I forgot to mention in the original post that I also removed old patch from net/remmina:

> svn delete files/patch-FreeRDP-Remmina-b6a55ae

Regards,
Maxim
Comment 2 muxas 2015-03-22 15:32:28 UTC
Hi,
Comment 3 John Marino freebsd_committer freebsd_triage 2015-03-27 13:24:09 UTC
Can you provide these diff set?  

E.g. as if ports was a svn repo and you did "cd net && svn diff --patch-compatible remmina remmina-plugin"

I can't read this archive (downloaded to windows lap) for some reason.


Also, note net/remmina was touched by yesterday, so patch should take that into account.
Comment 4 Pavel Timofeev 2015-12-02 18:36:30 UTC
Ports should be updated to 1.2 version, which is 1.2.0-rcgit5 now, because we already have freerdp 1.2 in ports tree.
I'll try to do it. Thanks to muxas@!
Comment 5 Pavel Timofeev 2015-12-09 12:12:37 UTC
Turns out, things are more complicated than I thought.
The latest "release" of freerdp project is already in our ports tree. It's marked with 1.2.0-beta1+android9 tag on Sep 12, 2014
https://github.com/FreeRDP/FreeRDP/tags

Of course, they didn't stop development. But they didn't made any tags or releases since then.

Meanwhile, the latest remmina tag is 1.2.0-rcgit.5.
https://github.com/FreeRDP/Remmina/tags
And it doesn't build against our freerdp version. No way.

I tried to build all of the remmina rcgit.# and managed to build only v1.2.0-rcgit.1 tag with one additional patch.

I've been using remmina a bit and found one problem with rdp plugin. It doesn't work well with 'Negotiate' auth. But freerdp from cmd works well with it. So, I beleive, this bug is already fixed somewhere in remmina code, but it needs a lot of time to dig git/bugs/pull_requests to find actual commit.

Maybe someone will find it in the future. Or they rather release freerdp and remmina 1.2.0 versions.

I make remmina-plugins-rdp work at least somehow now.
Comment 6 Pavel Timofeev 2015-12-09 12:17:08 UTC
remmina.patch is too big to upload it here.
You can download patch here: https://yadi.sk/d/52vYGsqXm6u57
Comment 7 Pavel Timofeev 2015-12-09 12:17:47 UTC
Created attachment 164012 [details]
net/remmina-plugins patch
Comment 8 Pavel Timofeev 2015-12-09 12:18:05 UTC
Created attachment 164013 [details]
net/remmina-plugin-rdp patch
Comment 9 Pavel Timofeev 2015-12-14 07:39:12 UTC
I think if we don't commit these changes (and waiting for real 1.2 release) we should extend expiration date of net/remmina-plugin-rdp.
http://www.freshports.org/net/remmina-plugin-rdp
Comment 10 VK freebsd_triage 2016-05-23 01:16:16 UTC
The original bug report no longer appears valid, and I'm considering this feedback timeout because it wasn't a proper diff patch and the reporter never addressed issues mentioned in Comment #3.

muxas, if you want to submit an updated, valid patch, please file a new bug report.

timp87, thank you for your efforts, but without a valid main remmina update patch, the update to plugins is redundant. Please consider filing a separate bug.