Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

chore: Release v4.0.0 #606

Merged
merged 12 commits into from
Dec 8, 2023
Merged

chore: Release v4.0.0 #606

merged 12 commits into from
Dec 8, 2023

Conversation

XavierChanth
Copy link
Member

- What I did

Cleaned up all remaining odds and ends:

  • Fixed path to dockerfiles which were moved in a previous PR
  • Updated the release build so that the copied files are flattened into the release zip (i.e. contents of core/shell/windows folders are in the root of the release zip)
  • Fixed up the install script
  • Added the debug sshrvd to the notarization script

- How I did it

- How to verify it

- Description for the changelog
chore: Release v4.0.0

@XavierChanth XavierChanth requested review from gkc and cpswan December 5, 2023 16:03
description: "Run x64 builds only"
required: true
default: false
type: boolean
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Added this to speed up the ability to confirm/test x64 releases, since non-x64 releases can take up to 10x longer

if is_root & ! [ -f "$user_bin_dir/$1" ] ; then
ln -sf "$dest/$1" "$user_bin_dir/$1"
echo "Linked $user_bin_dir/$1 to $dest"
fi
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

If the binaries were installed as root, we need to do a symbolic to where the binaries were installed, since we install the binaries to /usr/local/bin if we are root

dest="$user_bin_dir/$job_name.sh"
if ! [ -f "$dest" ]; then
cp "$script_dir/headless/$job_name.sh" "$dest"
fi
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Make necessary folders, and don't overwrite existing job scripts when reinstalling

pids=$(pgrep "$command")
if [ -n "$pids" ]; then
echo "$pids" | xargs kill
fi
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Only call the kill command if there are processes to kill

@gkc gkc merged commit e79a1dd into trunk Dec 8, 2023
11 checks passed
@gkc gkc deleted the release-v4.0.0 branch December 8, 2023 11:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants