Community Packages

Please read this before reporting a bug:
https://wiki.archlinux.org/title/Bug_reporting_guidelines

Do NOT report bugs when a package is just outdated, or it is in the AUR. Use the 'flag out of date' link on the package page, or the Mailing List.

REPEAT: Do NOT report bugs for outdated packages!
Tasklist

FS#18990 - [autojump] 8-1 missing zsh files in package

Attached to Project: Community Packages
Opened by John S. Smith (potatoe) - Wednesday, 07 April 2010, 18:43 GMT
Last edited by Dan Griffiths (Ghost1227) - Thursday, 08 April 2010, 21:11 GMT
Task Type Bug Report
Category Packages
Status Closed
Assigned To Dan Griffiths (Ghost1227)
Architecture All
Severity Low
Priority Normal
Reported Version
Due in Version Undecided
Due Date Undecided
Percent Complete 100%
Votes 0
Private No

Details

Description:
The autojump package is non-functional in zsh, due to missing files. In addition to not working in zsh, installing autojump also results in an error message on every login with zsh. This is because the pkg does not include the zsh support files, even though they are called from the included /etc/profile.d/autojump if zsh is running (and login shells always source /etc/profile).

I know the package is "Only guaranteed to work with bash" (post_install message), but considering upstream includes zsh support and considering that these missing files result in an error message for zsh users on every new login (e.g. ssh users), I think it's worth including the zsh files in the package. It's easier than patching /etc/profile.d/autojump to remove zsh support completely.

Additional info:
* package version(s)
autojump 8-1

Steps to reproduce:
After autojump 8-1 is installed, logging into zsh (or starting a new zsh with '/bin/zsh --login') will display the following error message:
/etc/profile.d/autojump.sh:.:20: no such file or directory: /etc/profile.d/autojump.zsh
and the 'j' command will not be added to the shell:
potatoebox% j
zsh: command not found: j

Fix:
Two files need to be added to the package. Modifying the PKGBUILD as follows will fix it:

--- a/PKGBUILD
+++ b/PKGBUILD
@@ -28,4 +28,6 @@ package() {
install -Dm644 autojump.1 ${pkgdir}/usr/share/man/man1/autojump.1
install -Dm755 autojump.bash ${pkgdir}/etc/profile.d/autojump.bash
install -Dm755 autojump.sh ${pkgdir}/etc/profile.d/autojump.sh
+ install -Dm755 autojump.zsh ${pkgdir}/etc/profile.d/autojump.zsh
+ install -Dm644 _j ${pkgdir}/usr/share/zsh/site-functions/_j
}
This task depends upon

Closed by  Dan Griffiths (Ghost1227)
Thursday, 08 April 2010, 21:11 GMT
Reason for closing:  Implemented
Additional comments about closing:  Thanks!

Loading...