Bug 150361 - [patch] provide script to bind with nautilus for multimedia/ffmpegthumbnailer
Summary: [patch] provide script to bind with nautilus for multimedia/ffmpegthumbnailer
Status: Closed FIXED
Alias: None
Product: Ports & Packages
Classification: Unclassified
Component: Individual Port(s) (show other bugs)
Version: Latest
Hardware: Any Any
: Normal Affects Only Me
Assignee: Stephen Montgomery-Smith
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2010-09-07 20:20 UTC by buganini
Modified: 2011-07-05 22:50 UTC (History)
0 users

See Also:


Attachments
file.diff (1.52 KB, patch)
2010-09-07 20:20 UTC, buganini
no flags Details | Diff
ddd (2.69 KB, text/plain)
2011-07-04 07:38 UTC, Stephen Montgomery-Smith
no flags Details

Note You need to log in before you can comment on or make changes to this bug.
Description buganini 2010-09-07 20:20:04 UTC
Provide a script that user can easily bind ffmpegthumbnailer with nautilus

Fix: Patch attached with submission follows:
Comment 1 Edwin Groothuis freebsd_committer freebsd_triage 2010-09-07 20:20:12 UTC
Maintainer of multimedia/ffmpegthumbnailer,

Please note that PR ports/150361 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/150361

-- 
Edwin Groothuis via the GNATS Auto Assign Tool
edwin@FreeBSD.org
Comment 2 Edwin Groothuis freebsd_committer freebsd_triage 2010-09-07 20:20:14 UTC
State Changed
From-To: open->feedback

Awaiting maintainers feedback (via the GNATS Auto Assign Tool)
Comment 3 Philip M. Gollucci freebsd_committer freebsd_triage 2010-09-23 06:18:25 UTC
State Changed
From-To: feedback->open

maintainer timeout (dierk@blaxxtarz.de ; 16 days)
Comment 4 Stephen Montgomery-Smith freebsd_committer freebsd_triage 2011-07-04 02:21:02 UTC
State Changed
From-To: open->feedback

I have questions for the submitter. 


Comment 5 Stephen Montgomery-Smith freebsd_committer freebsd_triage 2011-07-04 02:21:02 UTC
Responsible Changed
From-To: freebsd-ports-bugs->stephen

I'll take it.
Comment 6 Stephen Montgomery-Smith 2011-07-04 02:22:45 UTC
It seems to me that a problem with this patch is that the 
bind-nautilus.sh script is not available to anyone who installs this as 
a package.  So:

# pkg_add ffmpegthumbnailer-2.0.4_1.tbz

For Gnome user:
	If you wish to bind this with nautilus, see files/bind-nautilus.sh

The user installing this using a package doesn't have any idea where 
files/bind-nautilus.sh is.

Shouldn't this script be stored somewhere under /usr/local?  I must 
admit that I don't know where this sort of thing is conventionally stored.
Comment 7 buganini 2011-07-04 04:55:15 UTC
how about put it in share/ (%%DATADIR%%) ?
Comment 8 Stephen Montgomery-Smith 2011-07-04 07:38:02 UTC
I made some changes to your patch.  First, I have put the script into 
%%DATADIR%% as you suggest.  Second, I changed the typesetting of 
pkg-message.  portlint doesn't like blank lines at the end of 
pkg-message.  Thirdly, I added a comment to bind-nautilus.sh so that a 
casual user who comes across it will understand what it is about.

Tell me what you think.

I am a new committer, so after making the changes I will have to submit 
it to my mentors for approval.
Comment 9 buganini 2011-07-04 08:28:39 UTC
that's good, thank you!
Comment 10 dfilter service freebsd_committer freebsd_triage 2011-07-05 22:46:20 UTC
stephen     2011-07-05 21:46:07 UTC

  FreeBSD ports repository

  Modified files:
    multimedia/ffmpegthumbnailer Makefile pkg-plist 
  Added files:
    multimedia/ffmpegthumbnailer/files bind-nautilus.sh 
                                       pkg-message.in 
  Log:
  - Add script to allow users to bind with nautilus.
  - Bump portrevision.
  
  PR:             ports/150361
  Submitted by:   Kuan-Chung Chiu <buganini@gmail.com>
  Approved by:    maho (mentor)
  
  Revision  Changes    Path
  1.13      +7 -3      ports/multimedia/ffmpegthumbnailer/Makefile
  1.1       +16 -0     ports/multimedia/ffmpegthumbnailer/files/bind-nautilus.sh (new)
  1.1       +2 -0      ports/multimedia/ffmpegthumbnailer/files/pkg-message.in (new)
  1.5       +2 -0      ports/multimedia/ffmpegthumbnailer/pkg-plist
_______________________________________________
cvs-all@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/cvs-all
To unsubscribe, send any mail to "cvs-all-unsubscribe@freebsd.org"
Comment 11 Stephen Montgomery-Smith freebsd_committer freebsd_triage 2011-07-05 22:47:55 UTC
State Changed
From-To: feedback->closed

Committed, thanks!