gtorrent / gtorrent-gtk

Gtk client built on gtorrent-core
GNU General Public License v3.0
30 stars 9 forks source link

make install not working properly #98

Closed IGI-111 closed 10 years ago

IGI-111 commented 10 years ago

After installing gtorrent, the version in PATH crashes on start, probably due to missing dependencies.

igi-111 at Robb in ~/Work/gtorrent-gtk/build (master)
$ gtorrent-gtk
[2014-07-31 @ 14:55:09]: GTorrent v.0.0.2
[2014-07-31 @ 14:55:09]: Starting up core library...
[2014-07-31 @ 14:55:09]: The lock wasn't ready, retrying...
[2014-07-31 @ 14:55:09]: Process is unique
[2014-07-31 @ 14:55:09]: Process is unique
[2014-07-31 @ 14:55:09]: Process is unique
[2014-07-31 @ 14:55:09]: /home/igi-111/Downloads/ is the default save path
[2014-07-31 @ 14:55:09]: 5
[2014-07-31 @ 14:55:09]: 8
[2014-07-31 @ 14:55:09]: Starting up GUI layer...

** (gtorrent-gtk:28764): WARNING **: Can't load fallback CSS resource: Failed to import: La ressource dans « /org/gnome/adwaita/gtk-fallback.css » n'existe pas

** (gtorrent-gtk:28764): WARNING **: Can't load fallback CSS resource: Failed to import: La ressource dans « /org/gnome/adwaita/gtk-fallback.css » n'existe pas
[2014-07-31 @ 14:55:09]: -1
terminate called after throwing an instance of 'Glib::FileError'
[1]    28764 abort (core dumped)  gtorrent-gtk
fuyukaidesu commented 10 years ago

It works correctly in my wip/ui-rewrite branch, probably because resources are embedded into the executable.

IGI-111 commented 10 years ago

That's probably it then, it'll get fixed when wip/ui-rewrite is merged. Shall we close it now or wait until master is fine?

nyanpasu commented 10 years ago

@infoburp

That's probably it then, it'll get fixed when wip/ui-rewrite is merged. Shall we close it now or wait until master is fine?

It's a known issue. We'll leave it open anyway.

infoburp commented 10 years ago

I think it's best to leave open for now, @nyanpasu . I am now aware that it has been fixed in ui-rewrite though, that's great :)

How close is that to being ready to merge into master, @fuyukaidesu ?

nyanpasu commented 10 years ago

Really far away. It's so different from the master branch I guess we just started basing all our work on it. fuyukaidesu stopped "rewriting the ui" and just went straight to implementing RSS dialogs on ui/rewrite

infoburp commented 10 years ago

Oh dear, this isn't good, is there any way we could make a stable version of ui-rewrite, then merge that down before you drift too much further away from master?

infoburp commented 10 years ago

PS: I'm happy to put in a few hours this weekend + possibly Monday to get this done but I've got uni the rest of the week so I might not have much time.

nyanpasu commented 10 years ago

No. Let's just merge it now. (It's not like we have a team of lunatics who'd chew us out for it. It's stable enough.)

infoburp commented 10 years ago

Make it so.

fuyukaidesu commented 10 years ago

As @nyanpasu said, it's pretty stable, I haven't got a crash in 3 days, but as of right now, ui-rewrite relies on wip/rss, there's still a few things that needs to be done for the RSS dialog, the side bar, and implement labels before merging it into master.

But if we don't really care about master to have incomplete features, I can make a PR.

nyanpasu commented 10 years ago

@fuyukaidesu ww, I already merged it in.

Edit: gomen.

fuyukaidesu commented 10 years ago

Oops. just noticed.