[bug] Assumes all apps can be cloned from the `nextcloud` org (including suggested fix)
Nobody has claimed this yet.
Assessment
- Difficulty
- 2/5
- Estimated time
- 1-3 hours
- Newbie friendliness
- 78/100
Research direction
Start in bootstrap.sh at the linked line around 158 and trace how prompted extra apps are cloned. Run ./bootstrap.sh music before and after the change; done means the Music app is fetched from https://github.com/nc-music/music instead of being assumed to belong to the nextcloud organization.
Written by the indexing model from the issue text.
Description
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:
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
- Dominant language
- PHP
- Stars
- 190
- Forks
- 103
- Avg merge
- 4h 36m
- Merged PRs (30d)
- 1
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
More from nextcloud/nextcloud-docker-dev
-
bug
Difficulty 2/5 1-3 hours Newbie friendliness 72/100
nextcloud/nextcloud-docker-dev#483 · 1 comment · 3 reactions ·
-
bug
Difficulty 2/5 1-3 hours Newbie friendliness 70/100
nextcloud/nextcloud-docker-dev#480 · 1 comment ·
-
bug
Difficulty 3/5 1-2 days Newbie friendliness 55/100
-
Difficulty 3/5 1-2 days Newbie friendliness 45/100
-
Difficulty 3/5 1-2 days Newbie friendliness 45/100
nextcloud/nextcloud-docker-dev#446 · 1 comment · 1 reaction ·
All issues in nextcloud/nextcloud-docker-dev
Similar issues
-
priority: p3
Difficulty 2/5 1-3 hours Newbie friendliness 72/100
googleapis/librarian#7636 ·
-
0. Needs triage bug
Difficulty 2/5 1-3 hours Newbie friendliness 78/100
nextcloud/fulltextsearch#1011 ·
-
Difficulty 2/5 1-3 hours Newbie friendliness 84/100
-
Difficulty 2/5 1-3 hours Newbie friendliness 78/100
phpstan/phpstan-doctrine#794 ·
-
Difficulty 2/5 1-3 hours Newbie friendliness 74/100
Automattic/static-site-importer#1767 ·