Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fedora 26 erros #64

Open
piovisqui opened this issue Aug 25, 2017 · 7 comments
Open

Fedora 26 erros #64

piovisqui opened this issue Aug 25, 2017 · 7 comments

Comments

@piovisqui
Copy link

Hi folks.

When I try to run the current trunk it shows the error:

PAC started (pac) with PID 21828
INFO: Used config file '/home/piovisqui/.config/pac/pac.nfreeze'
Can't call method "get_color" on an undefined value at /opt/pac/lib/PACConfig.pm line 646.

The pac-4.5.5.7-2.x86_64.rpm has two errors:

This prevents xfreerdp from beaing used.
[WARN][com.freerdp.client.common.cmdline] - Using deprecated command-line interface!
[WARN][com.freerdp.client.common.compatibility] - -a 24 -> /bpp:24
[WARN][com.freerdp.client.common.compatibility] - -g 1309x1051 -> /size:1309x1051 or /w:1309 /h:1051
[WARN][com.freerdp.client.common.compatibility] - -u Administrator -> /u:Administrator
[WARN][com.freerdp.client.common.compatibility] - (nil) -> /v:(nil)
[WARN][com.freerdp.client.common.compatibility] - 
[ERROR][com.freerdp.client.x11] - error: server hostname was not specified with /v:<server>[:port]

This one can be solved by deleting the vte shipped with PAC.

Can't locate object method "new" via package "Gnome2::Vte::Terminal" (perhaps you forgot to load "Gnome2::Vte::Terminal"?) at /opt/pac/lib/PACTerminal.pm line 503.
*** ignoring at /opt/pac/lib/PACMain.pm line 1398.
 find /opt/pac/ -name "Vte.so*" -exec rm {} +

Others: https://sourceforge.net/p/pacmanager/bugs/247/

Any help?

@xenithorb
Copy link

Hi. I package pacmanager for Fedora and I use our packaged versions of vte... Can you try that out and see if it works for you?

@piovisqui
Copy link
Author

piovisqui commented Oct 16, 2017

Hi @xenithorb . Great job, it worked with vte and xfreerdp on Fedora 26.
@perseo22 Sell PAC to customers (maybe Red Hat?), it's just the best sysadmin tool out there :D

`
rpm -qa | grep -E "pacmanager|vte|xfreerdp"

vte291-0.48.3-1.fc26.x86_64
pacmanager-4.5.5.7-7.fc26.noarch
vte-profile-0.48.3-1.fc26.x86_64
vte-0.28.2-18.fc25.x86_64
`

@xenithorb
Copy link

Good news! Thanks for replying - There are some segfault bugs that I'm aware of but can't track down. If you encounter that please see if you can find a way to reproduce it.

@nerijus
Copy link

nerijus commented Nov 7, 2017

@xenithorb, are xfreerdp fixes in released Fedora package? It seems not.

@nerijus
Copy link

nerijus commented Nov 7, 2017

@xenithorb I managed to build asbru. Diff from the latest pacmanager.spec attached.

asbru.spec.diff.txt

@gfrenoy
Copy link

gfrenoy commented Nov 20, 2017

I did not test it but I don't believe the above spec file will work because it does not replace $RealBin correctly in "pac_conn". So some dependencies will fail when you'll start a new connection. See the latest version for a fix.

We use that spec file to publish our packaged version and it works now fine for both Fedora 26 and 27 :)

@nerijus
Copy link

nerijus commented Nov 20, 2017

I'd suggest adding these changes from my version:

--- asbru.spec.1        2017-11-21 01:20:31.718597897 +0200
+++ asbru.spec  2017-11-07 19:22:21.818593285 +0200
@@ -97,7 +58,7 @@
 mkdir -p %{buildroot}/{%{_mandir}/man1,%{_bindir}}
 mkdir -p %{buildroot}/%{_datadir}/{%{name}/{lib,res},applications}
 mkdir -p %{buildroot}/%{_bashcompletiondir}
-mkdir -p %{buildroot}/%{_datadir}/icons/hicolor/{24x24,64x64}/apps
+mkdir -p %{buildroot}/%{_datadir}/icons/hicolor/{24x24,64x64,256x256,scalable}/apps
 
 install -m 755 asbru %{buildroot}/%{_bindir}/%{name}
 install -m 755 utils/pac_from_mcm.pl %{buildroot}/%{_bindir}/%{name}_from_mcm
@@ -110,6 +71,8 @@
 # Copy the icons over to /usr/share/icons/
 cp -a res/asbru-logo-24.png %{buildroot}/%{_datadir}/icons/hicolor/24x24/apps/%{name}.png
 cp -a res/asbru-logo-64.png %{buildroot}/%{_datadir}/icons/hicolor/64x64/apps/%{name}.png
+cp -a res/asbru-logo-256.png %{buildroot}/%{_datadir}/icons/hicolor/256x256/apps/%{name}.png
+cp -a res/asbru-logo.svg %{buildroot}/%{_datadir}/icons/hicolor/scalable/apps/%{name}.svg
 
 # Copy the remaining resources and libraries
 cp -a res/*.{png,jpg,pl,glade} res/termcap %{buildroot}/%{_datadir}/%{name}/res/

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants