cerc-io/hosting
12
0
Fork 0
Code Issues 7 Pull Requests 3 Packages Projects Releases Wiki Activity

Add instructions for macOS. #70

Merged
dboreham merged 10 commits from telackey/osx into main 2023-10-18 20:06:28 +00:00
Conversation 0 Commits 10 Files Changed 3 +2 -1
1 changed files with 2 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files
Showing only changes of commit ade3e94f4a - Show all commits

3
gitea/initialize-gitea.sh
View File

@ -7,7 +7,8 @@ if [[ -n "$CERC_SCRIPT_DEBUG" ]]; then
fi
secure_password() {
# extra bytes so that even if we delete some chars we will still have plenty
# use openssl as the source, because it behaves similarly on both linux and macos
# we generate extra bytes so that even if tr deletes some chars we will still have plenty
openssl rand -base64 32 | tr -d '\/+=' | head -c 10 && echo
telackey marked this conversation as resolved Outdated
dboreham commented 2023-10-18 03:00:36 +00:00
Outdated
Review
Copy Link

Should also change the comment in line 9 since it no longer pertains?
(but probably this change should be in a separate PR?)

Should also change the comment in line 9 since it no longer pertains? (but probably this change should be in a separate PR?)
telackey commented 2023-10-18 03:16:41 +00:00
Outdated
Review
Copy Link

Done

Done
dboreham commented 2023-10-18 03:17:51 +00:00
Outdated
Review
Copy Link

Was the change necessary because macos doesn't have /dev/urandom ?

Was the change necessary because macos doesn't have `/dev/urandom` ?
telackey commented 2023-10-18 03:25:46 +00:00
Outdated
Review
Copy Link

No, it has /dev/urandom, but tr is not able to consume it without explicitly setting LC_CTYPE=C, otherwise it complains about invalid input. Once you get past that hurdle, the options do not behave the same way, and '-dc' ends up selecting the complement of the set rather than deleting it, so you end up with only unprintable characters.

No, it has /dev/urandom, but `tr` is not able to consume it without explicitly setting `LC_CTYPE=C`, otherwise it complains about invalid input. Once you get past that hurdle, the options do not behave the same way, and '-dc' ends up selecting the complement of the set rather than deleting it, so you end up with only unprintable characters.
dboreham commented 2023-10-18 03:39:34 +00:00
Outdated
Review
Copy Link

Might be worthwhile adding a comment saying openssl used because it's available and works on both macos and Linux?

Might be worthwhile adding a comment saying openssl used because it's available and works on both macos and Linux?
telackey commented 2023-10-18 03:45:04 +00:00
Outdated
Review
Copy Link

Added

Added
}