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

[DoomMake] extractUsedMapTextures() supports MAPINFO custom skies #99

Open
derekmd opened this issue Sep 21, 2023 · 2 comments
Open

[DoomMake] extractUsedMapTextures() supports MAPINFO custom skies #99

derekmd opened this issue Sep 21, 2023 · 2 comments
Assignees
Labels
enhancement Nice-to-haves and things that can actually help current behavior.

Comments

@derekmd
Copy link

derekmd commented Sep 21, 2023

Summary

I don't see any DoomTools code in this repo for parsing MAPINFO lumps so this may be out of the near-future scope. Would it be possible for the extractUsedMapTextures() function to include skies defined in map definitions of mapinfo lumps?

The current workaround is to apply the sky texture to at least one sidedef in a map.

Source port examples

  • UMAPINFO

    skytexture = "SOMESKY1"
    
  • DMAPINFO

    sky1 = "SOMESKY1"
    
  • EMAPINFO

    skyname = SOMESKY1
    sky2name = SOMESKY2
    altskyname = LNINGSKY
    
  • MAPINFO / ZMAPINFO

    Sky1 = "SOMESKY1"
    Sky2 = "SOMESKY1"
    

Links

@DRON12261
Copy link

MAPINFO in Hexen format (or ZDoom old format) still, it has a slightly different syntax:

sky1 SOMESKY1 <scrollspeed>
sky2 SOMESKY2 <scrollspeed>

https://zdoom.org/wiki/MAPINFO_(old_format)


Additionally, it's still possible to parse its format from Risen 3D and Doomsday, and RMAPINFO from Doom Retro (deprecated format, now it use UMAPINFO), but I don't know if anyone is using it today or not.
https://manual.dengine.net/ded/map_info
https://github.com/bradharding/doomretro/wiki/Information-for-Mappers#MAPINFO-Lumps

@MTrop
Copy link
Owner

MTrop commented Sep 24, 2023

If I were to add this, I'd want it to be done in WTexScan, which would pick up "used" textures in a WAD or individual map. WTEXport will then just see it as another texture, so I'd have to do nothing to get the export working.

@MTrop MTrop self-assigned this Sep 24, 2023
@MTrop MTrop added the enhancement Nice-to-haves and things that can actually help current behavior. label Sep 24, 2023
@MTrop MTrop removed their assignment Sep 25, 2023
@MTrop MTrop self-assigned this Dec 8, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement Nice-to-haves and things that can actually help current behavior.
Projects
None yet
Development

No branches or pull requests

3 participants