chaotic-aur / toolbox

Unified kit with all the scripts required for maintaining the repository 🧰
https://aur.chaotic.cx
GNU Affero General Public License v3.0
81 stars 11 forks source link

[Discussion] Base Singularity on base-devel instead of base-master #59

Closed Gontier-Julien closed 1 year ago

Gontier-Julien commented 1 year ago

I propose some change on the Singularity file for the toolbox, i don't know why it was setup that way (and i think there was a very good reason behind it, so clear me up if i'm wrong). But since it in a clean docker anyway, there shouldn't be any issues to go directly grab the base-devel docker container. And with that we could remove the pacman line to go grab it.

From: gitlab.archlinux.org:5050/archlinux/archlinux-docker:base-master pacman -Syyu --noconfirm --noprogressbar --quiet pacman -S --noconfirm --noprogressbar --quiet base base-devel

to

From: gitlab.archlinux.org:5050/archlinux/archlinux-docker:base-devel pacman -Syyu --noconfirm --noprogressbar --quiet

PedroHLC commented 1 year ago

i don't know why it was setup that way (and i think there was a very good reason behind it, so clear me up if i'm wrong).

Actually, neither @thotypous nor I was aware of the existence of the base-devel image when we ported it to singularity.

Feel free to open a PR to keep your credits.

Gontier-Julien commented 1 year ago

i don't know why it was setup that way (and i think there was a very good reason behind it, so clear me up if i'm wrong).

Actually, neither @thotypous nor I was aware of the existence of the base-devel image when we ported it to singularity.

Feel free to open a PR to keep your credits.

Alright doing that right now!