Wrong symlink path of ponythink #98

Closed
svenstaro opened this Issue Sep 9, 2012 · 12 comments

Comments

Projects
None yet
3 participants
Collaborator

svenstaro commented Sep 9, 2012

Currently in the Arch package, the ponythink symlink that ends up in /usr/bin/ points to /build/pkg/usr/bin/ponysay which is the absolute path inside the chroot it is build in. Why not just make it a relative path to just ponysay? They are never meant to be in separate dirs anyway. The version this appeared in is 2.5.1.

Collaborator

jristz commented Sep 9, 2012

what is the form tah ponythink is simlinkes?
the direct path or build inside the ./setup.py??

I make a ln -s ./pony inside the dir for the pony simlinking, probably (not testes) a ln -s ./ponythink ./ponysay in the dir make the simlink and modify the setup to manage it as same way as other symlinks can work

the "Bad way" is a little setup that call ponysay with the proper changes and this script called ponythink
and the "really bad way" is duplicate the ponysay with ponythink name

more ideas??

Collaborator

svenstaro commented Sep 9, 2012

Sorry, I can't really understand what you are saying. The best way of doing this would be:

cd $DESTDIR/$PREFIX/bin/
ln -s ponysay ponythink
Collaborator

jristz commented Sep 9, 2012

stupid translator never work as spect XD

I try to say first if the ponythink is simlinked inside the setup.py or in the "make"
second I ask if a already existant simlink like all in ponies/ now, can work as spected (a simlink point to ./ponysay)

the others are not so-practicals forms

Collaborator

svenstaro commented Sep 10, 2012

I use setup.py. And yes, a premade symlink should work.

Collaborator

maandree commented Sep 11, 2012

I have fixed this.

@maandree maandree closed this Sep 11, 2012

Collaborator

svenstaro commented Sep 11, 2012

Can we get a release that includes this soon?

Collaborator

maandree commented Sep 11, 2012

done

Collaborator

maandree commented Sep 11, 2012

@svenstaro
bash is not dependency for ponysay, you should remove it for the Arch repo.

coreutils is a dependency but is transversally required by python:
python → openssl → perl → coreutils

Collaborator

jristz commented Sep 11, 2012

probably the bash is more like a makedepend for the ./dependency-test.sh

Note: and for the info I submit ponysay-git and util-say{normal and git} to the ccr (chakra community repo aka chakra aur) if you want add to the readme/readme.md/ponysay.texinfo or not

Collaborator

maandree commented Sep 11, 2012

I'll add CCR as soon as possible, is any addition information needed, or
is it just as simple as with AUR?

I will add distribution info for util-say, probabily under info for contributors;
or I will compile an other list for all related packages in a new section.

Collaborator

maandree commented Sep 11, 2012

OK, I see that it is just like AUR.

In your ponysay-git (i have mentioned this in aur, but he has not fixed it)
optdepends=('util-say: tty2colourfultty and ponysay2ttyponysay')
Should be made more descriptive, for example:
optdepends=('util-say: Improved TTY support utilising KMS and support for PNG files')
Also, python3-distribute will not be needed until issue #94 is resolved.

Collaborator

jristz commented Sep 11, 2012

ok I submit to ccr the new one now and I remain in Arch but Chakra need ponies too
And now I summit the stable 2.6 too

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment