dependency is broken in krdc #11

ericklzr commented 2014年07月18日 20:39:31 +02:00 (Migrated from github.com)

Krdc uses freerdp now and no more rdesktop.
erro-krdc

Krdc uses freerdp now and no more rdesktop. 

demmm commented 2014年07月18日 21:22:57 +02:00 (Migrated from github.com)

Thanks for reporting, please test with:
sudo pacman -U http://kaosx.us/repo/build/freerdp-1.2.0beta-1-x86_64.pkg.tar.xz
sudo pacman -U http://kaosx.us/repo/build/kdenetwork-krdc-4.13.3-2-x86_64.pkg.tar.xz
If you don't use the [build] repo, if working correctly will move to the [apps] repo shortly.

Thanks for reporting, please test with: sudo pacman -U http://kaosx.us/repo/build/freerdp-1.2.0beta-1-x86_64.pkg.tar.xz sudo pacman -U http://kaosx.us/repo/build/kdenetwork-krdc-4.13.3-2-x86_64.pkg.tar.xz If you don't use the [build] repo, if working correctly will move to the [apps] repo shortly.

ericklzr commented 2014年07月21日 15:09:21 +02:00 (Migrated from github.com)

The issue of dependence is OK. Now the problem is the parameters:

The new freerdp uses:

Usage: xfreerdp [file] [options] [/v:[:port]]

Syntax:
/flag (enables flag)
/option: (specifies option with value)
+toggle -toggle (enables or disables toggle, where '/' is a synonym of '+')

/v:<server>[:port] Server hostname
/port:<number> Server port
/w:<width> Width
/h:<height> Height
/size:<width>x<height> Screen size
/f Fullscreen mode

...
I believe we need to change the rdpview.cpp where

...
arguments << "-g" << QString::number(width) + 'x' + QString::number(height); # "/size:"
arguments << "-k" << m_hostPreferences->keyboardLayout(); # "/kdb:

 if (!m_url.userName().isEmpty())
 arguments << "-u" << m_url.userName(); # "/u:"
 else
 arguments << "-u" << ""; # "/u:"
 if (!m_url.password().isNull())
 arguments << "-p" << m_url.password(); # "/p:"

...

The issue of dependence is OK. Now the problem is the parameters: The new freerdp uses: Usage: xfreerdp [file] [options] [/v:<server>[:port]] Syntax: /flag (enables flag) /option:<value> (specifies option with value) +toggle -toggle (enables or disables toggle, where '/' is a synonym of '+') ``` /v:<server>[:port] Server hostname /port:<number> Server port /w:<width> Width /h:<height> Height /size:<width>x<height> Screen size /f Fullscreen mode ``` ... I believe we need to change the rdpview.cpp where ... arguments << "-g" << QString::number(width) + 'x' + QString::number(height); # "/size:" arguments << "-k" << m_hostPreferences->keyboardLayout(); # "/kdb: ``` if (!m_url.userName().isEmpty()) arguments << "-u" << m_url.userName(); # "/u:" else arguments << "-u" << ""; # "/u:" if (!m_url.password().isNull()) arguments << "-p" << m_url.password(); # "/p:" ``` ...

demmm commented 2014年07月21日 15:17:50 +02:00 (Migrated from github.com)

Those are not packaging issues, but rather need to be taken upstream.
Did you file a bug report with either freerdp or krdc?

Those are not packaging issues, but rather need to be taken upstream. Did you file a bug report with either freerdp or krdc?

ericklzr commented 2014年07月21日 15:35:12 +02:00 (Migrated from github.com)

Problem in krdc. The new freerdp uses different parameters, and krdc is sending to xfreerdc deprecated parameters. Excuse the horrible English. I speak Portuguese.

Problem in krdc. The new freerdp uses different parameters, and krdc is sending to xfreerdc deprecated parameters. Excuse the horrible English. I speak Portuguese.

demmm commented 2015年01月20日 22:33:51 +01:00 (Migrated from github.com)

krdc 14.12.1 had an upstream fix for this.
Please report back if working correctly now.

krdc 14.12.1 had an upstream fix for this. Please report back if working correctly now.

demmm commented 2015年02月11日 02:07:41 +01:00 (Migrated from github.com)

No further response from submitter

No further response from submitter