Open parmentf opened 11 years ago
À mon avis (je reviens au français, on est dans pkgi, que diable!) on pourrait ajouter ce petit bout de code à pkgi/nodejs/bin/pkgi-install-nodejs
:
# Fix syntax problems
mv <?php echo getenv('APPNAME_HOME') ?>/nvm/nvm.sh <?php echo getenv('APPNAME_HOME') ?>/nvm/to_fix_nvm.sh
cat <?php echo getenv('APPNAME_HOME') ?>/nvm/to_fix_nvm.sh | \
sed -e 's/function rc_nvm_version {/rc_nvm_version () {/' | \
sed -e 's/if \[\[ "\$PATTERN" == v[?][*]\.[?][*]\.[?][*] \]\];/if [ "$PATTERN" = "v?*.?*.?*" ];/' | \
sed -e 's/if \[\[ -d "\$NVM_DIR\/\$VERSION" \]\];/if [ -d "$NVM_DIR\/$VERSION" ];/' | \
> <?php echo getenv('APPNAME_HOME') ?>/nvm/nvm.sh
Par contre, je n'ai pas refait un test complet pour voir si ça ne casse rien.
Oui tant qu'a faire si tu as corrigé le problème, on pourrait simplement forker le dépôt git de nvm et appliquer ton patch dessus. Ensuite pkgi-install-nodejs n'aurait plus qu'a se baser sur ce fork patché. Ca évite ainsi les sed dans le code qui alourdissent sa lisibilité.
On Thu, Jan 31, 2013 at 3:51 PM, François Parmentier < notifications@github.com> wrote:
À mon avis (je reviens au français, on est dans pkgi, que diable!) on pourrait ajouter ce petit bout de code à pkgi/nodejs/bin/pkgi-install-nodejs:
Fix syntax problems
mv <?php echo getenv('APPNAME_HOME') ?>/nvm/nvm.sh <?php echo getenv('APPNAME_HOME') ?>/nvm/to_fix_nvm.sh cat <?php echo getenv('APPNAME_HOME') ?>/nvm/to_fix_nvm.sh | \ sed -e 's/function rc_nvm_version {/rc_nvmversion () {/' | \ sed -e 's/if [[ "\$PATTERN" == v[?][].[?][].[?][] ]];/if [ "$PATTERN" = "v?.?.?_" ];/' | \ sed -e 's/if [[ -d "\$NVM_DIR\/\$VERSION" ]];/if [ -d "$NVM_DIR\/$VERSION" ];/' | \
<?php echo getenv('APPNAME_HOME') ?>/nvm/nvm.sh
Par contre, je n'ai pas refait un test complet pour voir si ça ne casse rien.
— Reply to this email directly or view it on GitHubhttps://github.com/kerphi/pkgi/issues/8#issuecomment-12945661.
J'ai forké nvm, j'ai testé à la main: on peut lancer apache et nodejs en utilisant ce fork. Je pushe les modifs permettant de prendre en compte le bon nvm. On verra plus tard pour le tag et le site pkgi.net?
When installing only apache2, php, and nodejs, you can't start apache2. Probable cause: nvm bad bash syntax (nvm is installed from the last version in github, it may have been working before, not no more now).
Here is my
pkgi.env
:nodejs
module usesnvm
, and add some inclusions in etc/profile.d, which crash the etc/init.d apache2 script.