Bug 96149 - [patch] Make sysutils/bpm handle OPTIONS
Summary: [patch] Make sysutils/bpm handle OPTIONS
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: Sam Lawrance
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2006-04-21 19:10 UTC by Panagiotis Astithas
Modified: 2006-05-14 16:00 UTC (History)
0 users

See Also:


Attachments
bpm.patch (1.34 KB, patch)
2006-04-21 19:10 UTC, Panagiotis Astithas
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Panagiotis Astithas 2006-04-21 19:10:20 UTC

sysutils/bpm cannot handle ports with OPTIONS, since it launches a dialog process in a shell invisible to the user. This makes the operation appear like halted. This patch is probably the simplest solution to the OPTIONS handling problem. Using gdialog (from x11/zenity) we get a popup window with the specified options, although without correct initial values. Nevertheless, this way the user has a chance to build or upgrade a port without the process hanging in the background, waiting for user input, making bpm usable for every use case I've come across.

How-To-Repeat: 

Try to use bpm to build/update a port that has OPTIONS in the Makefile.
Comment 1 Edwin Groothuis freebsd_committer freebsd_triage 2006-04-21 19:12:10 UTC
State Changed
From-To: open->feedback

Awaiting maintainers feedback
Comment 2 Seth Kingsley 2006-04-21 19:46:38 UTC
Looks good.  The only thing I want to avoid is unwanted GNOME linkage.
Currently BPM has no forced GNOME-libs dependencies.  Can zenity be built with
GTK+ only?
Comment 3 Panagiotis Astithas 2006-05-09 15:43:55 UTC
Doh, I never received your followup for some reason.

Looking at the zenity port it appears that it will bring along a few 
low-level GNOME stuff (ORBit2, gconf2, gnomehier, etc.), not the real 
heavyweights (gnomemedia, gnomelibs, gnomeapplets, etc.). Naturally, 
users that get bpm as a gnome2-power-tools dependency won't care, but 
others might. Unfortunately I haven't found anything similar, but 
DE-agnostic.

Since bpm is a desktop application, I personally don't mind a little 
extra cruft every now and then in exchange for unique functionality. 
Perhaps this should be considered a temporary measure, until proper 
OPTIONS handling is implemented in bpm without extra helper applications.
Comment 4 Sam Lawrance freebsd_committer freebsd_triage 2006-05-13 13:41:20 UTC
State Changed
From-To: feedback->feedback

Seth, what do you think?  Is it OK with the few extra 
dependencies zenity requires?
Comment 5 Sam Lawrance freebsd_committer freebsd_triage 2006-05-14 04:57:10 UTC
State Changed
From-To: feedback->open

Feedback received, dep on zenity is OK. 


Comment 6 Sam Lawrance freebsd_committer freebsd_triage 2006-05-14 04:57:10 UTC
Responsible Changed
From-To: freebsd-ports-bugs->lawrance

Take.
Comment 7 Sam Lawrance freebsd_committer freebsd_triage 2006-05-14 16:00:16 UTC
State Changed
From-To: open->closed

Committed, thanks!