Opened 7 years ago

Closed 7 years ago

Last modified 7 years ago

#42741 closed update (fixed)

grilo/grilo-plugins: update to latest versions

Reported by: dbevans (David B. Evans) Owned by: juanrgar (Juan R. García Blanco)
Priority: Normal Milestone:
Component: ports Version:
Keywords: Cc:
Port: grilo grilo-plugins

Description

$ port livecheck grilo*
grilo seems to have been updated (port version: 0.2.8, new version: 0.2.9)
grilo-plugins seems to have been updated (port version: 0.2.9, new version: 0.2.11)

Attachments (2)

Portfile-grilo.diff (844 bytes) - added by juanrgar (Juan R. García Blanco) 7 years ago.
grilo Portfile
Portfile-grilo-plugins.diff (2.4 KB) - added by juanrgar (Juan R. García Blanco) 7 years ago.
grilo-plugins Portfile

Download all attachments as: .zip

Change History (16)

comment:1 Changed 7 years ago by juanrgar (Juan R. García Blanco)

Thank you. I was aware of this; I hadn't enough time to port these new versions, sorry. I'll post patches here asap.

comment:2 Changed 7 years ago by juanrgar (Juan R. García Blanco)

I managed to build both grilo@0.2.9 and grilo-plugins@0.2.11. However I get a segfault (11) when trying to run any test application, i.e. grilo-simple-playlist, grl-inspect-0.2, grilo-test-ui. I think you got exactly this behavior in previous versions. I've run them through gdb, but wherever it crashes, there there are no debug symbols, so I see nothing. I'm trying now using lldb. I'll give it a try tomorrow and upload patches for current Portfiles.

comment:3 Changed 7 years ago by dbevans (David B. Evans)

To get symbols you need to build with

configure.optflags -O0 -g

and then use the port's -k option to stop it from deleting the source files after installation completes. A suitable build command to upgrade (rebuild) an already installed port would be

sudo port -nsk upgrade --force grilo configure.optflags='-O0 -g'

Do the same for grilo-plugins and any dependencies for which you want symbols.

gdb can be then used in the normal manner.

Last edited 7 years ago by dbevans (David B. Evans) (previous) (diff)

comment:4 Changed 7 years ago by dbevans (David B. Evans)

FYI, GNOME 3.11.91 (3.12 beta2) was released this week and the final 3.12 release is scheduled 3 weeks hence on March 25. I'm testing 3.11.91 now and hope to upgrade trunk to 3.12 sometime in early April. So if there's anything you want to include in the pre-update testing, it would help if you could submit it in the next week or so. Thanks.

comment:5 Changed 7 years ago by juanrgar (Juan R. García Blanco)

Mm, I was using configure.cflags-append -O0 -g, and then clean{} to prevent clean operation, which is kind of a mess. I prefer your solution indeed.

I've no ports pending to be submitted; I'd like to finally submit nemiver (although I'm not sure if it's part of gnome); however, right now I don't have much spare time.

I forgot to mention that grilo-plugins now includes a pocket plugin that depends on goa 3.11; I've disabled it as I don't have installed that version of goa.

comment:6 Changed 7 years ago by dbevans (David B. Evans)

You don't want to use configure.cflags-append because configure.optflags defaults to -Os. So want to replace that entirely. See edited version of comment:3 for how to do it on the command line without editing the Portfile.

comment:7 Changed 7 years ago by juanrgar (Juan R. García Blanco)

Thank you :)

I managed to run the three grilo utilities (grilo-simple-playlist says 'poll(2) failed due to: Bad file descriptor.' though). I had to disable the dmap plugin, which was causing the crash. I don't know why. If I can't manage to write a patch in a reasonable amount of time, then I think we could just leave it disabled.

comment:8 Changed 7 years ago by juanrgar (Juan R. García Blanco)

Well, this is kind of funny. This is my analysis of why grilo is crashing when the dmap plugin is enabled.

The crash occurs in grl_dmap_plugin_init() (https://git.gnome.org/browse/grilo-plugins/tree/src/dmap/grl-dmap.c) in line 147, when accessing the 'error' object. libdmapsharing is build with --with-mdns=dns_sd, so the code of dmap_mdns_browser_start() is in https://git.gnome.org/browse/libdmapsharing/tree/libdmapsharing/dmap-mdns-browser-dnssd.c. As you can see, that function always returns FALSE besides not allocating a GError object.

We could patch grl_dmap_plugin_init() to not check the error object returned by dmap_mdns_browser_start(), but I think that would solve nothing. I don't know why libdmapsharing is being compiled with dns_sd, but I assume there's a good reason. Therefore, I would upgrade grilo and grilo-plugins, and disable the dmap plugin, at least for now, until we find a solution for libdmapsharing.

comment:9 Changed 7 years ago by juanrgar (Juan R. García Blanco)

I'll upload then the patches. I didn't make any changes to the grilo Portfile; and for grilo-plugins I just disabled dmap, since I inspected the ChangeLog and I found no new plugins added since the last release.

Changed 7 years ago by juanrgar (Juan R. García Blanco)

Attachment: Portfile-grilo.diff added

grilo Portfile

Changed 7 years ago by juanrgar (Juan R. García Blanco)

Attachment: Portfile-grilo-plugins.diff added

grilo-plugins Portfile

comment:10 Changed 7 years ago by dbevans (David B. Evans)

Thanks for figuring this all out. I'll take a look at the dmap issues over the weekend but disabling the plugin for now seems like a good idea.

comment:11 Changed 7 years ago by dbevans (David B. Evans)

Resolution: fixed
Status: newclosed
  • grilo update committed in r117697
  • grilo-plugins committed with modifications in r117699 (update m4/intltool.m4 before autoreconf to fix intltool configuration, build deps, add missing avahi dep, --enable-freebox)
  • grilo, grilo-plugins copied to GNOME-3/unstable test branch in r117700 (increment revisions, enable pocket plugin)

comment:12 Changed 7 years ago by juanrgar (Juan R. García Blanco)

I know this is closed, but have you ported gnome-logs. I had that on my pending list. If you haven't, I'd like to give it a try this afternoon.

comment:13 Changed 7 years ago by juanrgar (Juan R. García Blanco)

Well, apparently gnome-logs is designed for systemd. So I guess it does not worth porting neither systemd nor gnome-logs.

comment:14 Changed 7 years ago by dbevans (David B. Evans)

I haven't looked at it. Does it really require systemd or is that just optional and recommended (several other ports in this last category)?

Note: See TracTickets for help on using tickets.