Summary: | [PATCH] multimedia/ffmpegthumbnailer is missing gconf2 depend when GNOME OptionsNG has been selected | ||||||||
---|---|---|---|---|---|---|---|---|---|
Product: | Ports & Packages | Reporter: | cheunghonyu | ||||||
Component: | Individual Port(s) | Assignee: | Jason E. Hale <jhale> | ||||||
Status: | Closed FIXED | ||||||||
Severity: | Affects Only Me | ||||||||
Priority: | Normal | ||||||||
Version: | Latest | ||||||||
Hardware: | Any | ||||||||
OS: | Any | ||||||||
Attachments: |
|
Description
cheunghonyu
2012-10-07 19:40:09 UTC
Maintainer of multimedia/ffmpegthumbnailer, Please note that PR ports/172459 has just been submitted. If it contains a patch for an upgrade, an enhancement or a bug fix you agree on, reply to this email stating that you approve the patch and a committer will take care of it. The full text of the PR can be found at: http://www.freebsd.org/cgi/query-pr.cgi?pr=ports/172459 -- Edwin Groothuis via the GNATS Auto Assign Tool edwin@FreeBSD.org State Changed From-To: open->feedback Awaiting maintainers feedback (via the GNATS Auto Assign Tool) Yes I'm agree. That dependency has to be here. State Changed From-To: feedback->open Maintainer approved. Responsible Changed From-To: freebsd-ports-bugs->jhale I'll take it. There were a few other changes that I needed to make. Could you review the attached patch? - Trimmed Makefile header - Dropped ABI version numbers from LIB_DEPENDS to satisfy portlint(1) - USE_GNOME= pkgconfig -> USE_PKGCONFIG= build - Removed soon to be deprecated WANT_GNOME - Install GCONF_SCHEMAS in PREFIX instead of LOCALBASE I've tested it, and it looks nice and works like before ;). I've done all dirty job, thank you very much. Is it normal to install gnome schemas like this: gconftool-2 --makefile-install-rule in post-install: ? I didn't see anything like this in any other port. It was my improvisation. So, if it is normal, please commit it. On Friday, October 12, 2012 00:54:21 paranormal wrote: > > Is it normal to install gnome schemas like this: > gconftool-2 --makefile-install-rule in post-install: ? > > I didn't see anything like this in any other port. > It was my improvisation. > > So, if it is normal, please commit it. > I looked at how bsd.gnome.mk handles gconf schemas and I think this is the way to go: .if ${PORT_OPTIONS:MGNOME} ${MKDIR} ${PREFIX}/etc/gconf/schemas ${INSTALL_DATA} ${FILESDIR}/${GCONF_SCHEMAS} ${PREFIX}/etc/gconf/schemas @${SETENV} GCONF_CONFIG_SOURCE=xml:${GCONF_CONFIG_OPTIONS}:${PREFIX}/${GCONF_CONFIG_DIRECTORY} \ gconftool-2 --makefile-install-rule ${PREFIX}/etc/gconf/schemas/${GCONF_SCHEMAS} \ > /dev/null || ${TRUE} .endif I'm more of a KDE guy, but this code seems to work fine on tinderbox. The problem with `gconftool-2 --get-default-source` is that it provides LOCALBASE instead of PREFIX. https://redports.org/buildarchive/20121012041037-24617/ - Jason I'm disagree. I'd like to see string: install blablabla ffmpegthumbnailer.schemas in installing output. That was the reason why I had been deleting /dev/null || ${TRUE} But if your insist: no problem. oups, sorry. I was scared by red colour of string I thought it something
new.
> /dev/null || ${TRUE}
please commit your patch.
State Changed From-To: open->closed Committed. Thanks! |