Skip to content

Build with ocicl instead of git submodules (first-pass)#3661

Draft
logoraz wants to merge 1 commit intoatlas-engineer:masterfrom
logoraz:master
Draft

Build with ocicl instead of git submodules (first-pass)#3661
logoraz wants to merge 1 commit intoatlas-engineer:masterfrom
logoraz:master

Conversation

@logoraz
Copy link
Contributor

@logoraz logoraz commented Jun 29, 2025

Description

This PR is a first pass at updating the build process to use ocicl to pull the cl systems needed instead of git submodules.
Opening for review and better visibility to align/clean things up accordingly. Happy to help/assist an any way necessary, just need advisement.

Update makefile to incorporate build with ocicl

  • First pass at incorporating ocicl into build (needs refinement)
  • Update makefile replacing submodules flag with ocicl flag
  • Updated gitignore to ignore ocicl directory & asdf registry init.lisp
  • Kept _build directory for later removal
  • Did not include ocicl package registry ocicl.csv for now (this is where we would pin specific commits)
    • As of current this build will install latest of all cl systems

Fixes # (issue) - N/A

Checklist:

I have only tested this build with the electron variant as of current - would need assistance checking the below:

  • Git branch state is mergable.
  • Changelog is up to date (via a separate commit).
  • New dependencies are accounted for.
  • Documentation is up to date.
  • Compilation and tests ((asdf:test-system :nyxt/<renderer>))
    • No new compilation warnings.
    • Tests are sufficient.

  - First pass at incorporating ocicl into build (needs refinement)
  - Update makefile replacing submodules flag with ocicl flag
  - Updated gitignore to ignore ocicl directory & registry ocicl.csv
  - Kept _build directory for later removal
@logoraz logoraz marked this pull request as draft June 29, 2025 23:06
@jmercouris jmercouris changed the title Build with ocicl instead of git submodulels (first-pass) Build with ocicl instead of git submodules (first-pass) Jul 16, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

1 participant