noobs icon indicating copy to clipboard operation
noobs copied to clipboard

Published 20 hours ago • verified publisher icon unkn0w

GitActions - testowanie wykonywalności plików

Open loper opened this issue 3 years ago • 6 comments

Czy w takim GitActions da się dodać krok sprawdzający, czy nowy skrypt ma mode 0755? To by pilnowało, żeby wszystko było wykonywalne. @mikasjp

loper avatar Jan 26 '22 21:01 loper

Czy dobrze rozumiem, że chcesz, żeby Git pamiętał uprawienia plików ze skryptami? Poczytałem trochę na ten temat i wydaje mi się, że to nie jest dobry pomysł.

  • Git nie ma takiej możliwości "out of the box". Wymaga to dodatkowej konfiguracji.
  • Co jeśli ktoś pisze skrypt i używa Gita pod innym OS, który ma odmienny system uprawnień (np. Windows)?
  • Czy nie lepiej zmodyfikować skrypt https://noobs.mikr.us w taki sposób, żeby po sklonowaniu repo nadawał odpowiednie uprawnienia?

mikasjp avatar Jan 30 '22 12:01 mikasjp

Chciałbym, aby jako 1 z kroków testowania było wykonywane sprawdzanie, czy plik posiada tryb wykonywalny.

loper avatar Jan 30 '22 21:01 loper

Chyba nie do końca rozumiem, co masz na myśli. Podczas testów są nadawane odpowiednie uprawnienia: https://github.com/unkn0w/noobs/blob/53f65269cf53f2d1cccc374be230106f6b10e69e/tests/execute_new_and_modified_scripts.sh#L16 Zaraz po sklonowaniu repozytorium skrypty nie będą wykonywalne.

mikasjp avatar Feb 07 '22 20:02 mikasjp

Scenariusz:

  • User dodaje nowy skrypt.
  • Zapomina jednak o tym, co większość - o nadaniu flagi wykonywalności (+x).
  • Test w Actions wyłapuje taki błąd i od razu sypie błędem.

loper avatar Feb 07 '22 20:02 loper

https://github.com/mati97xdd/noobs/commit/2f155df1298c7dc389fc921cb7e807cbd2d1a9f8 Tak będzie ok?

mati97xdd avatar Feb 15 '22 12:02 mati97xdd

Z samego czytania kodu - dokładnie o coś takiego mi chodziło. Da się to podpiąć do obecnych testów? Dzięki, L.

loper avatar Feb 15 '22 13:02 loper