tcsh: Update to version 6.19.00, csh is now a symlink to tcsh #83
Conversation
| mee_install() { | ||
| bee_install | ||
| ( cd "${D}"/"${BEE_BINDIR}"; ln -sv tcsh csh ) | ||
| ( cd "${D}"/"${MANDIR}/man1"; ln -sv tcsh.1 csh.1 ) |
There was a problem hiding this comment.
Why not put it into mee_install_post?
There was a problem hiding this comment.
why not symlink via ln -sv tcsh ${D}${BINDIR}/csh ? no subshell, no cd
There was a problem hiding this comment.
I think it's a matter of personal preference. Is the csh link part of the installation or not?
Is install_post for fixups of the build or for fixups belonging to mariux?
There was a problem hiding this comment.
'why not symlink...':
requires more brain, I'm never shure if your proposal is 'amd'-proof. The subshell way seems to me bullet-proof.
There was a problem hiding this comment.
I think it's a matter of personal preference. Is the csh link part of the installation or not?
Is install_post for fixups of the build or for fixups belonging to mariux?
It is indeed a matter of taste. I prefer mee_install_post as you do not have to repeat the bee_install line then.
|
I'd prefer On the other hand, there is more than one way to do it and the other variants produce the same result. So to me they are valid, too, and could have been merged as well. We can try to home in on a common style, but this shouldn't be a blocker for an update. |
No description provided.