User Details
User Details
- User Since
- Jun 3 2023, 6:16 AM (16 w, 5 d)
Jun 7 2023
Jun 7 2023
charlesthobe added a comment to D152064: CI: run "apt update" once per job before installing packages.
I think this is ready for review unless you see problems in it?
Jun 4 2023
Jun 4 2023
charlesthobe added a comment to D152064: CI: run "apt update" once per job before installing packages.
these were just comments on why I don't have a check for ubuntu in this area, nothing needs to be done.
charlesthobe added inline comments to D152064: CI: run "apt update" once per job before installing packages.
charlesthobe added inline comments to D152064: CI: run "apt update" once per job before installing packages.
charlesthobe added a comment to D152064: CI: run "apt update" once per job before installing packages.
I apologize for the too many revisions, and that the initial patch wasn't in a production ready state, but it's now ready for merging, I believe.
Jun 3 2023
Jun 3 2023
charlesthobe updated the diff for D152064: CI: run "apt update" once per job before installing packages.
charlesthobe updated the diff for D152064: CI: run "apt update" once per job before installing packages.
charlesthobe updated the diff for D152064: CI: run "apt update" once per job before installing packages.
charlesthobe updated the diff for D152064: CI: run "apt update" once per job before installing packages.
Use "apt update" before "llvm/actions/install-ninja" which uses "apt install" without "apt update" beforehand
Apt update is now its own step for ease of readability and a less hacky looking solution with running it within the first step which installs packages
charlesthobe added a comment to D152064: CI: run "apt update" once per job before installing packages.
One second please I have an improvement to make
charlesthobe retitled D152064: CI: run "apt update" once per job before installing packages from CI: run "apt update" before installing any packages to CI: run "apt update" once per job before installing packages.
charlesthobe added a comment to D152064: CI: run "apt update" once per job before installing packages.
commit message should be:
CI: run "apt update" once per job before installing packages
imo
charlesthobe added a comment to D152064: CI: run "apt update" once per job before installing packages.
charlesthobe added a comment to D152064: CI: run "apt update" once per job before installing packages.
charlesthobe updated the diff for D152064: CI: run "apt update" once per job before installing packages.
removed double "sudo"
charlesthobe added a comment to D152064: CI: run "apt update" once per job before installing packages.
there's a double sudo, one second while I fix it.
charlesthobe requested review of D152064: CI: run "apt update" once per job before installing packages.