FS#20669 - [git] package installs gitk but not gitk's dependencies

Attached to Project: Arch Linux
Opened by Jan Dlabal (y27) - Monday, 30 August 2010, 01:13 GMT
Last edited by Dan McGee (toofishes) - Monday, 05 September 2011, 12:48 GMT
Task Type Bug Report
Category Packages: Extra
Status Closed
Assigned To Dan McGee (toofishes)
Architecture All
Severity Low
Priority Normal
Reported Version
Due in Version Undecided
Due Date Undecided
Percent Complete 100%
Votes 6
Private No

Details

Description:
This issue arrised in the forum post here:
https://bbs.archlinux.org/viewtopic.php?pid=817963

In my opinion including gitk in the git package while making the dependencies necessary for gitk (tcl, tk) optional in the git package does not make sense.

I think that optional dependencies should be used only for parts of applications that are not necessarily needed -- as an example let's say you are installing an editor and there is an optional dependency which is a plugin to enable syntax colorization for some language.

However, it doesn't make sense to even install gitk if it can't be used with the minimal dependencies -- I assume that if I don't get gitk : command not found, it should be able to start.

For this reason, (not only) I think that it would be more logical and intuitive to provide a separate gitk package where tcl and tk would be required dependencies, and a git package without gitk.
This task depends upon

Closed by  Dan McGee (toofishes)
Monday, 05 September 2011, 12:48 GMT
Reason for closing:  Won't implement
Additional comments about closing:  Don't reopen a bug I've made a subjective decision on. You're not going to change my mind here.
Comment by Jan Dlabal (y27) - Wednesday, 01 September 2010, 23:50 GMT
Attaching two patches I made, PKGBUILD.nogui and PKGBUILD.gui.

What I would suggest is to use PKGBUILD.nogui for the current git package, which would therefore only install non-GUI components of git and therefore would not need tcl/tk.

The second patch PKGBUILD.gui, which should be used for a git with gui package, say git-gui. Everything else could just be copied from the current git package.
Comment by Dan McGee (toofishes) - Wednesday, 01 September 2010, 23:54 GMT
Definitely don't need (and I won't even touch this with) two PKGBUILDs. Please read about split packages and try again and I will be a lot more likely to make this change.

I also disagree with your characterization of optdepends; we also don't split packages just for kicks around here, especially if depends aren't being dragged in and you are expected to read pacman output.
Comment by Robert Meijers (RobertMe) - Thursday, 02 September 2010, 15:57 GMT
The attached patch builds two packages, git and gitk. Git is the normal package but without gitk (and of course doesn't have tk as optional dependency), the gitk package only installs gitk and has tk and git as dependency.
Comment by Eric Belanger (Snowman) - Wednesday, 09 February 2011, 09:35 GMT
Status? Do we close this as "Not a bug" / "Won't fix"?
Comment by Dan McGee (toofishes) - Wednesday, 09 February 2011, 13:57 GMT
I'm just not feeling the need for a split package here; I don't want to become Debian/Ubuntu/every other distro that supersplits packages.
Comment by flagos (flagos) - Saturday, 03 September 2011, 13:06 GMT
To me, this explanation doesn't make sense.

You say: "I'm just not feeling the need for a split package here;"

That's your opinion, I can consider that 2 package rather than only one can be a better solution (I don't mind).

Problem is that gitk has been included and not tcl/tk package that are required dependencies to run gitk.

Have a look at the issue discussed https://bbs.archlinux.org/viewtopic.php?pid=817963 , error message returned by gitk is just incomprehensible ;-)
Comment by Allan McRae (Allan) - Saturday, 03 September 2011, 13:15 GMT
We need four packages! Split out the components of "git svn" and "git send-email" into separate packages while you are at it as they do not run without installing the required optdepends...
Comment by Evangelos Foutras (foutrelis) - Saturday, 03 September 2011, 13:16 GMT
Well, it doesn't sound like an outrageous proposal, although I don't mind the single package either.

@flagos: I'm going to disagree with you regarding the error message; it's not cryptic. When confronted with "wish: not found", an Archer's next step should be to find the package which provides this 'wish' command thingy. Or google the error message. Or read https://wiki.archlinux.org/index.php/Git. :)

Loading...