Bug 123257 - [PATCH] editors/vim options does not work as expected
Summary: [PATCH] editors/vim options does not work as expected
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: David E. O'Brien
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2008-04-30 13:40 UTC by Mel
Modified: 2008-05-01 16:40 UTC (History)
1 user (show)

See Also:


Attachments
file.diff (529 bytes, patch)
2008-04-30 13:40 UTC, Mel
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Mel 2008-04-30 13:40:01 UTC
When building vim with WITH_VIM_OPTIONS and not adding *any* gui, the GNOME1
gui is added regardless. There is also no option in the vim options to set
the equivalent of WITHOUT_X11.

Fix: The hack below implements the correct logic. Really what's needed is 'NO_GUI'
or 'WITHOUT_X11' option in the vim options.
How-To-Repeat: Set WITH_VIM_OPTIONS in /etc/make.conf, type make config in
/usr/ports/editors/vim, do not check any GUI, including the one advertized as
GNOME1, then type make -V LIB_DEPENDS to see that gtk-1 dep has been added.

Here's the current /var/db/ports/vim/options that demonstrates the case:

# This file is auto-generated by 'make config'.
# No user-servicable parts inside!
# Options for vim-7.1.242
_OPTIONS_READ=vim-7.1.242
WITH_PERL=true
WITHOUT_PYTHON=true
WITHOUT_RUBY=true
WITH_CSCOPE=true
WITHOUT_EXUBERANT_CTAGS=true
WITHOUT_ATHENA=true
WITHOUT_GTK2=true
WITHOUT_GNOME=true
WITHOUT_MOTIF=true
WITHOUT_XTERM_SAVE=true
Comment 1 Edwin Groothuis freebsd_committer freebsd_triage 2008-04-30 14:20:17 UTC
Responsible Changed
From-To: freebsd-ports-bugs->obrien

Over to maintainer (via the GNATS Auto Assign Tool)
Comment 2 David E. O'Brien freebsd_committer freebsd_triage 2008-04-30 18:45:40 UTC
State Changed
From-To: open->feedback

sent counter-patch
Comment 3 Mel 2008-04-30 20:33:59 UTC
Counter patch works as expected. Of course, it requires people to read and not 
just "check GTK2", but that's perfectly fine with me ;)
-- 
Mel
Comment 4 Mark Linimon freebsd_committer freebsd_triage 2008-04-30 22:52:19 UTC
State Changed
From-To: feedback->open

From misfiled PR ports/123272: 

Date: Wed, 30 Apr 2008 10:45:25 -0700 
Comment 5 David E. O'Brien freebsd_committer freebsd_triage 2008-05-01 16:31:46 UTC
On Wed, Apr 30, 2008 at 09:33:59PM +0200, Mel wrote:
> Counter patch works as expected. Of course, it requires people to read
> and not just "check GTK2", but that's perfectly fine with me ;)

If you think it should default on "ON" that would be OK with me.
Comment 6 dfilter service freebsd_committer freebsd_triage 2008-05-01 16:38:02 UTC
obrien      2008-05-01 15:37:57 UTC

  FreeBSD ports repository

  Modified files:
    editors/vim          options 
  Log:
  If one wanted to build a GUI-less, one got stuck with GNOME1 anyway.
  So add the "X11" knob so 'WITHOUT_X11' will properly get set.
  
  PR:             123257, 123272
  Reported by:    Mel <mel@rachie.is-a-geek.net>
  
  Revision  Changes    Path
  1.2       +1 -0      ports/editors/vim/options
_______________________________________________
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 7 David E. O'Brien freebsd_committer freebsd_triage 2008-05-01 16:38:10 UTC
State Changed
From-To: open->closed

committed