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

feat: Move libpointmatcher build-system logic to norlab-build-system submodule (clean-up and polish) #545

Conversation

RedLeader962
Copy link
Collaborator

@RedLeader962 RedLeader962 commented Dec 14, 2023

Summary:

Reference task NMO-398 in YouTrack board

Refactor out generic logic that can be reuse accros NorLab repositories from libpointmatcher-build-system to norlab-build-system library.

Changes and type of changes:

  • fix: solve the problem with lpm_crawl_dependencies_build_matrix.bash not passing argument properly to sub-shell
    • source script instead of run in sub-shell
    • add bats test
    • add script to run bats test in N2ST
  • fix: refactor tests related to tests_docker_interactive/, update run configuration setup, restructure directories
  • refactor: Rename run configurations
  • refactor: strip SITREP related logic in dotenv files, docker compose files and dockerfiles
  • refactor: execute NBS docker install and buildx create related logic instead of re-implementing them. Add bats test
  • build: bump NBS submodule version
  • chore: update run config

@RedLeader962 RedLeader962 merged commit ef48562 into develop Dec 14, 2023
4 checks passed
@RedLeader962 RedLeader962 changed the title Move libpointmatcher build-system logic to norlab-build-system submodule (clean-up and polish) feat: Move libpointmatcher build-system logic to norlab-build-system submodule (clean-up and polish) Dec 22, 2023
@RedLeader962 RedLeader962 mentioned this pull request Dec 22, 2023
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