On 28/09/15 11:44, Sam Thursfield wrote:
On 26/09/15 08:35, Paul Sherwood wrote:
> PS I took a shot at a patch...
Thanks! I think this is a good idea in principle. This means that adding
strata to systems will now involve writing this:
- morph: baserock/strata/input-common.morph
instead of this:
- morph: strata/input-common.morph
I think that's actually better because it makes it clear that you are
including a stratum maintained by the Baserock project in your system.
And if you want you can copy it to myproject/strata/input-common.morph
instead, which would mean that you're 'forking' it effectively.
The patch needs to update the paths in all system and stratum .morph
files, at the moment nothing at all builds :-)
If you or someone else can do a patch with all paths updated, I think
should go for it...
...actually, that would create a lot of pain for people who are already
working in a fork of definitions.git. They will have added new system
.morph files, and those will break when they merge this patch into their
repo, because their systems will still contain the old paths
We could write an automated migration fairly easily that looked at all
.morph files and added 'baserock/' to each path. Maybe it would be
enough to send an email to baserock-announce@ linking to the script and
suggesting that everyone should run it in their clone of definitions.git
as soon as they get the chance.
Sam Thursfield, Codethink Ltd.
Office telephone: +44 161 236 5575