Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fichier .rotrrent.rc écrasé lors de mise à jour #72

Closed
Merrick28 opened this issue Jan 17, 2023 · 4 comments
Closed

Fichier .rotrrent.rc écrasé lors de mise à jour #72

Merrick28 opened this issue Jan 17, 2023 · 4 comments

Comments

@Merrick28
Copy link

Bonjour,

lors d'une mise à jour (par exemple via watchtower), le fichier .rtorrent.rc est écrasé, et donc supprime les réglages qu'on a pu faire (notamment la variable directory.default.set, ce qui peut être gênant...).

Il faudrait que ce fichier ne soit pas touché lors du démarrage du container, ou bien que l'on puisse mettre en variables les chemins choisis

@Rathorian
Copy link
Collaborator

Bonjour,

J'imagine que c'est du à ma récente modification #68
Je devais faire la modification dans le .rtorrent.rc pour prendre en compte les changements.
Malgré tout, j'ai mis en place une copie de sauvegarde du fichier d'origine dans /config/rtorrent/.rtorrent.rc.old

Aurais-tu une idée pour faire différemment ?

@Merrick28
Copy link
Author

Merrick28 commented Jan 17, 2023

Pour moi le plus gênant est le répertoire de destination, le reste n'est pas vital. Du coup, le plus simple serait de mettre en place une nouvelle variable au lancement du torrent (DOWNLOAD_DIRECTORY) et de faire un sed comme tu l'as fait dans le commit indiqué au dessus, en gardant la variable /data/downloads par défaut si cette variable n'est pas settée

Autre solution possible pour éviter la multiplication des variables, dans le startup, quand tu testes l'existence du fichier (ligne 89), tu peux aussi faire un grep de <PORT_DHT> dans ce fichier. Si tu le trouves : le fichier est vierge, on peut intervenir, sinon, le fichier a déjà été configuré, et on ne fait rien.

if [ -e /config/rtorrent/.rtorrent.rc ]; then
  if grep -q '<PORT_DHT>' config/rtorrent/.rtorrent.rc; then
    
      mv /config/rtorrent/.rtorrent.rc /config/rtorrent/.rtorrent.rc.old
      mv /home/torrent/.rtorrent.rc /config/rtorrent/.rtorrent.rc
      ln -s /config/rtorrent/.rtorrent.rc /home/torrent/.rtorrent.rc
  else
   # fichier déjà present
   ;
  fi
else
  mv /home/torrent/.rtorrent.rc /config/rtorrent/.rtorrent.rc
  ln -s /config/rtorrent/.rtorrent.rc /home/torrent/.rtorrent.rc
fi


(edit pour typo)

@Rathorian
Copy link
Collaborator

On va rester sur ta première solution.
Je ferais la modification un peu plus tard dans la journée

Rathorian added a commit that referenced this issue Jan 17, 2023
add(): variable (DOWNLOAD_DIRECTORY) #72
@Merrick28
Copy link
Author

Testé et validé ! Merci beaucoup :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants