git/list[1] front-page[2] threads[3] people[4] search[5] about
 

Re: [PATCH] Add --with-tcltk and --without-tcltk to configure.

From
EREygene Ryabinkin <rea-git@codelabs.ru>
Date
Mar 29, 2007, 08:58 UTC
Message-ID
<20070329085834.GJ59098@codelabs.ru>
In-Reply-To
<7v7it04fbo.fsf@assigned-by-dhcp.cox.net>
Thu, Mar 29, 2007 at 01:35:55AM -0700, Junio C Hamano wrote:
Show 7 quoted lines
> Eygene Ryabinkin <rea-git@codelabs.ru> writes:
> 
> > Or you meant something different? I am little confused with
> > the '$PATH' in your example. Was it intended?
> 
> Yes, the dollar-sign-in-pathname is pretty much a part of my
> pathological example. 

Got it. So you mean that the '$' sign should be escaped as well? ;)) Or we can get another _SQ substitution to the script and the string will look like exec 'whatever you'\''d written here' "$0" -- "$@"

Show 17 quoted lines
> >> But come to think of it, it lets shell handle $PATH to find wish
> >> anyway, so *unless* we have specific version dependency to wish
> >> that wish binary normally found on user's $PATH is inadequate,
> >> we probably should not even need to be doing any of this path
> >> munging.  You might end up discovering the path to wish binary
> >> in your autoconf script, we do not have to use it.  ./configure
> >> can just see if there is wish, and set NO_TCLTK appropriately
> >> without any of the path business.
> >> 
> >> What do you think?
> >
> > There are problems at least with FreeBSD: it just installs the
> > wish8.4, wish8.3, wish8.2, etc. It does not provide the bare 'wish'
> > as the link to one of those.
> 
> Then sed -e 's/wish/$(WISH_NAME)/', still letting the shell to
> handle the path part, could be a simpler option.  I dunno.

Ah, you mean that './configure --with-tcltk=wish8.4' should also do the trick? It seems to be easy to achieve by just skipping the 'test -x' part in the configure.ac. So the semantics of '--with-tcltk=PATH' will be: "If you're telling me about the path to the interpreter, it is you who should take care of it. I do not mind if you will give me something unexecutable, unexistent and so on.". Comments?

-- 
Eygene
Previous: Junio C HamanoNext: Tom Prince
Message 42 of 47 in “Added make options NO_GUI and WITH_P4IMPORT.”
  1. Added make options NO_GUI and WITH_P4IMPORT.Eygene Ryabinkin, Mar 20, 2007
  2. Jakub NarebskiMar 21, 2007
  3. Eygene RyabinkinMar 21, 2007
  4. Johannes SchindelinMar 21, 2007
  5. Eygene RyabinkinMar 21, 2007
  6. Johannes SchindelinMar 21, 2007
  7. Paolo BonziniMar 21, 2007
  8. Eygene RyabinkinMar 21, 2007
  9. Paolo BonziniMar 21, 2007
  10. Alex RiesenMar 21, 2007
  11. Jakub NarebskiMar 24, 2007
  12. Eygene RyabinkinMar 21, 2007
  13. Johannes SchindelinMar 21, 2007
  14. Eygene RyabinkinMar 21, 2007
  15. Junio C HamanoMar 21, 2007
  16. Eygene RyabinkinMar 26, 2007
  17. Added options NO_TCLTK, WITH_P4IMPORT and --with-tcltk/--without-tcltk.Eygene Ryabinkin, Mar 26, 2007
  18. Junio C HamanoMar 26, 2007
  19. Add the WITH_P4IMPORT knob to the Makefile.Eygene Ryabinkin, Mar 27, 2007
  20. Junio C HamanoMar 27, 2007
  21. Eygene RyabinkinMar 27, 2007
  22. Added git-p4 package to the list of git RPMs.Eygene Ryabinkin, Mar 27, 2007
  23. Remove unused WITH_OWN_SUBPROCESS_PY from RPM specBrian Gernhardt, Mar 27, 2007
  24. Junio C HamanoApr 4, 2007
  25. Eygene RyabinkinApr 5, 2007
  26. Jakub NarebskiMar 26, 2007
  27. Junio C HamanoMar 26, 2007
  28. Eygene RyabinkinMar 26, 2007
  29. Junio C HamanoMar 27, 2007
  30. Eygene RyabinkinMar 27, 2007
  31. Added configure options --with-tcltk/--without-tcltk.Eygene Ryabinkin, Mar 27, 2007
  32. Junio C HamanoMar 27, 2007
  33. Eygene RyabinkinMar 27, 2007
  34. Junio C HamanoMar 28, 2007
  35. Add --with-tcltk and --without-tcltk to configure.Eygene Ryabinkin, Mar 28, 2007
  36. Added Tcl/Tk interpreter path rewriting for the GUI tools.Eygene Ryabinkin, Mar 28, 2007
  37. Junio C HamanoMar 28, 2007
  38. Eygene RyabinkinMar 29, 2007
  39. Junio C HamanoMar 29, 2007
  40. Eygene RyabinkinMar 29, 2007
  41. Junio C HamanoMar 29, 2007
  42. Eygene RyabinkinMar 29, 2007
  43. Tom PrinceMar 29, 2007
  44. Eygene RyabinkinMar 29, 2007
  45. Eliminate checks of user-specified Tcl/Tk interpreter.Eygene Ryabinkin, Mar 29, 2007
  46. Added correct Python path to the RPM specfile.Eygene Ryabinkin, Mar 29, 2007
  47. Junio C HamanoMar 26, 2007

Read the whole thread, see it on lore, or plain text.

$ cat FOOTERMessages come from the public archive at lore.kernel.org/git, fetched every hour. The front page is chosen and written each morning by an AI editor and can be wrong; the threads themselves are the record. About and API. For agents: an MCP server at https://gitlist.dev/mcp, and any thread, story or person page as Markdown by adding .md to its URL (or sending Accept: text/markdown). Details in /llms.txt.