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

configure: some way to extend xml across the asset stack #1056

Open
bates64 opened this issue May 15, 2023 · 0 comments
Open

configure: some way to extend xml across the asset stack #1056

bates64 opened this issue May 15, 2023 · 0 comments
Labels
build system Relates to build system internals like configure.py enhancement New feature or request question Further information is requested

Comments

@bates64
Copy link
Member

bates64 commented May 15, 2023

Builds on from #1053. Much lower priority I think.

Imagine we have the asset stack us, mymod and these two files:

<!-- assets/us/sprite/npc/AlbinoDino/SpriteSheet.xml -->
<xml>
    <Rasters>
        <Raster file="Raster1.png" />
    </Rasters>
</xml>
<!-- assets/mymod/sprite/npc/AlbinoDino/SpriteSheet.extend.xml -->
<xml>
    <Rasters>
        <Raster file="Raster2.png" />
    </Rasters>
</xml>

The result is an XML that contains both Rasters, rather than just Raster2 if mymod used SpriteSheet.xml instead of SpriteSheet.extend.xml.

Specifically, I'm thinking Configure.resolve_asset_path/similar can automatically look for ".extend" versions of appropriate file types. This will implement some kind of merging/extending heuristic that enables asset directories (mods) to add new content / replace existing content without redeclaring any data from the originally rom.

We can extend this to other files types perhaps, e.g. #1032 uses YAML right now

@bates64 bates64 added the enhancement New feature or request label May 15, 2023
@bates64 bates64 added the build system Relates to build system internals like configure.py label Jan 7, 2024
@bates64 bates64 added the question Further information is requested label May 25, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
build system Relates to build system internals like configure.py enhancement New feature or request question Further information is requested
Projects
None yet
Development

No branches or pull requests

1 participant