[bug] Assumes all apps can be cloned from the `nextcloud` org (including suggested fix)

Aperta Adatta ai principianti
#476 0 commenti 0 reazioni 0 assegnatari Vedi su GitHub

Nessuno ha ancora preso questa issue.

Valutazione

Difficoltà
2/5
Tempo stimato
1-3 ore
Idoneità per principianti
78/100
Tipo di issue
Bug
Chiarezza
Specificata chiaramente
Stato di attività
Tranquilla
Stack tecnologico
git, shell
Ambito
devops

Direzione di ricerca

Inizia in bootstrap.sh alla riga indicata intorno alla 158 e segui il modo in cui vengono clonate le app aggiuntive richieste. Esegui ./bootstrap.sh music prima e dopo la modifica; il lavoro è completato quando l’app Music viene recuperata da https://github.com/nc-music/music invece di assumere che appartenga all’organizzazione nextcloud.

Scritto dal modello di indicizzazione a partire dal testo della issue.

Descrizione

The Music app's repo is at https://github.com/nc-music/music but the script assumes all extra apps prompted by the user are under the nextcloud org, so running ./bootstrap.sh music fails. The following is the line doing this:

https://github.com/juliusknorr/nextcloud-docker-dev/blob/0cbb0767b210e523e679fa405db639e822a07af7/bootstrap.sh#L158


Maybe supporting a list of known apps can help, this also makes it easier to contribute extra apps. For example:

function clone_app() {
  TARGET=workspace/server/apps-extra/"$1"
  if [ -d "$TARGET"/.git ]; then
    echo "🆗 App $1 is already installed." | indent
    return
  fi
  (
    echo "🌏 Fetching $1"
    (git clone ${APPS_CLONE_PARAMS[@]+"${APPS_CLONE_PARAMS[@]}"} "$2".git "$TARGET" 2>&1 | indent_cli &&
      echo "✅ $1 installed") ||
      echo "❌ Failed to install $1"
  ) | indent
}

function install_app() {
  case "$1" in
    viewer)             clone_app $1 "https://github.com/orgs/nextcloud/viewer" ;;
    recommendations)    clone_app $1 "https://github.com/orgs/nextcloud/recommendations" ;;
    files_pdfviewer)    clone_app $1 "https://github.com/orgs/nextcloud/files_pdfviewer" ;;
    profiler)           clone_app $1 "https://github.com/orgs/nextcloud/profiler" ;;
    hmr_enabler)        clone_app $1 "https://github.com/orgs/nextcloud/hmr_enabler" ;;
    circles)            clone_app $1 "https://github.com/orgs/nextcloud/circles" ;;
    globalsiteselector) clone_app $1 "https://github.com/orgs/nextcloud/globalsiteselector" ;;
    music)              clone_app $1 "https://github.com/nc-music/music" ;;
    onlyoffice)         clone_app $1 "https://github.com/ONLYOFFICE/onlyoffice-nextcloud" ;;
    *)
      cat << EOF
The '$1' app is not recognized by this script. If it is not a typo please open an issue to include it.

Valid extra apps to install:
  music
  onlyoffice
EOF
      exit 1
      ;;
  esac
}

...

for app in "${APPS_TO_INSTALL[@]}"
do
  install_app "$app"
done
Lingua principale
PHP
Stelle
190
Fork
103
Merge medio
4h 36m
PR unite (30g)
1

Guida per i contributori

Apri la guida per i contributori

Come iniziare

  1. Leggi tutta la issue e poi la guida ai contributi del progetto.
  2. Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
  3. Fai un fork del repository e lavora su un branch.
  4. Apri una pull request che faccia riferimento al numero della issue.

Altre issue di nextcloud/nextcloud-docker-dev

Tutte le issue di nextcloud/nextcloud-docker-dev

Issue simili

Altre issue su PHP

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.