Skip to content

Update Docker file for package-based LLVM/Clang install#183

Merged
abeltrano merged 3 commits into
developfrom
updatedocs
Feb 28, 2024
Merged

Update Docker file for package-based LLVM/Clang install#183
abeltrano merged 3 commits into
developfrom
updatedocs

Conversation

@abeltrano
Copy link
Copy Markdown
Contributor

Type

  • Bug fix
  • Feature addition
  • Feature update
  • Documentation
  • Build Infrastructure

Side Effects

  • Breaking change
  • Non-functional change

Goals

  • Minimize Docker image build time and simplify server controller setup, taking advantage of LLVM and Clang 17 now being available in the main package repository for Ubuntu 23.10 mantic.

Technical Details

  • Replace manual llvm installation script with use of clang-*-17 and llvm-*-17 packages.
  • Update Linux README for how to set up a development machine using package-based LLVM+Clang installation.
  • Fix fenced code appearing in lists in Linux README.
  • Replace use of apt-get <command> with apt <command>.
  • Ad apt upgrade to bring packages to newest versions.

Test Results

  • Built and published both docker images netremote-build and netremote-dev` on Windows.
  • Set up a new server afresh, followed the Linux README instructions by copy-and-pasting the commands, then pulled and built the project successfully.

Reviewer Focus

  • None

Future Work

  • None

Checklist

  • Build target all compiles cleanly.
  • clang-format and clang-tidy deltas produced no new output.
  • Newly added functions include doxygen-style comment block.

@abeltrano abeltrano requested a review from a team as a code owner February 28, 2024 03:37
@abeltrano abeltrano merged commit 8dcf1bb into develop Feb 28, 2024
@abeltrano abeltrano deleted the updatedocs branch February 28, 2024 04:18
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.

1 participant