[Git][archlinux/packaging/packages/electron27][main] 2 commits: Use rbash for generated script to reduce chance of anything going sideways

[Caleb Maclennan (@alerque)]

Caleb Maclennan pushed to branch main at Arch Linux / Packaging / Packages / 
electron27


Commits:
ccbba22c by Caleb Maclennan at 2024-03-04T12:59:50+03:00
Use rbash for generated script to reduce chance of anything going sideways

- - - - -
d7146c1a by Caleb Maclennan at 2024-03-04T12:59:56+03:00
Always remove targets during source prep so build can run outside of chroots

- - - - -


2 changed files:

- PKGBUILD
- makepkg-source-roller.py


Changes:

=====================================
PKGBUILD
=====================================
@@ -232,7 +232,7 @@ sha256sums=('SKIP'
             '1808df5ba4d1e2f9efa07ac6b510bec866fa6d60e44505d82aea3f6072105a71'
             '893bc04c7fceba2f0a7195ed48551d55f066bbc530ec934c89c55768e6f3949c'
             'ff588a8a4fd2f79eb8a4f11cf1aa151298ffb895be566c57cc355d47f161f53f'
-            '1eebf52f298ffb0a5525fa64b28039a6a0b5d83c07c3457262c88e9cc4bb0451'
+            '3ae82375ba212c31fd4ba6f1fa4e2445eeca8eb8c952176131ad57c0258db224'
             'SKIP'
             'SKIP'
             'SKIP'
@@ -428,7 +428,7 @@ prepare() {
   echo "Putting together electron sources"
   # Generate gclient gn args file and prepare-electron-source-tree.sh
   python makepkg-source-roller.py generate electron/DEPS $pkgname
-  bash prepare-electron-source-tree.sh "$CARCH"
+  rbash prepare-electron-source-tree.sh "$CARCH"
   mv electron src/electron
 
   echo "Running hooks..."


=====================================
makepkg-source-roller.py
=====================================
@@ -215,7 +215,7 @@ def generate_source_list(deps, indent, extra_sources, 
pkgname, reverse_map):
 
 
 def generate_managed_scripts(deps, extra_cmds, pkgname, reverse_map):
-    script = """#!/bin/bash
+    script = """#!/usr/bin/env rbash
 set -e
 # Generated file. Do not modify by hand.
 # Usage: script <CARCH>
@@ -225,8 +225,8 @@ place_subproject_into_tree () {
     if [[ -n "$parent_dir" ]]; then
         mkdir -p "$parent_dir"
     fi
-    # Remove the target dir only if it's empty
-    rm -df "$2"
+    # Remove the target dir
+    rm -rf "$2"
     if [[ "$3" == "true" ]]; then
         cp -r "$1" "$2"
     else



View it on GitLab: 
https://gitlab.archlinux.org/archlinux/packaging/packages/electron27/-/compare/0193bc40aa25e61f5b06de5a1ca9a95eb2a86c59...d7146c1a097bf15d48e13001e326a06a9ac86f38

-- 
View it on GitLab: 
https://gitlab.archlinux.org/archlinux/packaging/packages/electron27/-/compare/0193bc40aa25e61f5b06de5a1ca9a95eb2a86c59...d7146c1a097bf15d48e13001e326a06a9ac86f38
You're receiving this email because of your account on gitlab.archlinux.org.