Skip to content

Conversation

@thorsten-klein
Copy link
Contributor

@thorsten-klein thorsten-klein commented Oct 17, 2025

Proposal for #615

Support is added for a new import-modifications section in the west manifest.

Under the url-replace key, users can define search-and-replace patterns that are applied to project URLs during manifest import. This allows downstream projects to modify remote URLs, e.g. when using mirrored repositories.

@thorsten-klein thorsten-klein marked this pull request as draft October 17, 2025 13:07
@codecov
Copy link

codecov bot commented Oct 17, 2025

Codecov Report

❌ Patch coverage is 95.65217% with 1 line in your changes missing coverage. Please review.
✅ Project coverage is 84.53%. Comparing base (a4ce10a) to head (bc81ddf).

Files with missing lines Patch % Lines
src/west/manifest.py 95.65% 1 Missing ⚠️
Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##             main     #869      +/-   ##
==========================================
+ Coverage   84.40%   84.53%   +0.13%     
==========================================
  Files          11       11              
  Lines        3385     3408      +23     
==========================================
+ Hits         2857     2881      +24     
+ Misses        528      527       -1     
Files with missing lines Coverage Δ
src/west/manifest.py 95.53% <95.65%> (+0.19%) ⬆️

@thorsten-klein thorsten-klein marked this pull request as ready for review October 26, 2025 16:01
@thorsten-klein thorsten-klein changed the title [draft] Allow modifying remote/project urls of imported manifests Allow modifying remote/project urls of imported manifests Oct 26, 2025
thorsten-klein and others added 2 commits October 28, 2025 17:51
Add support for a new 'import-modifications' section in the manifest.

Under the 'url-replace' key, users can define search-and-replace
patterns that are applied to project URLs during manifest import.
This allows downstream projects to modify remote URLs, e.g. when using
mirrored repositories.
Test that import-modifications feature works to override west project
urls that directly specified in west manifest or imported via
submanifests.
@marc-hb
Copy link
Collaborator

marc-hb commented Nov 5, 2025

Do you think you could focus on the west config approach first? Because it should be smaller and easier to experiment with, review, test and use in anger etc., while sharing the code with any manifest-based approach later. As discussed in #615, imports can be very tricky and very error-prone.

Obviously, your #867 will make this even easier.

@thorsten-klein thorsten-klein marked this pull request as draft November 6, 2025 08:42
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.

2 participants