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

Include bootstrap IWADs? #7

Open
Voros2 opened this issue Jul 16, 2017 · 6 comments
Open

Include bootstrap IWADs? #7

Voros2 opened this issue Jul 16, 2017 · 6 comments

Comments

@Voros2
Copy link
Contributor

Voros2 commented Jul 16, 2017

Deutex had a bootstrap IWAD in its distribution, users won't need any other IWAD to run Deutex. There might be a few exceptions, such as Strife or Hexen, but maybe that can be addressed Freedoom v0.9 bootstrap IWAD and Zauberer bootstrap IWAD could be used.

@chungy
Copy link
Contributor

chungy commented Jul 16, 2017

I don't know why you'd use the Freedoom 0.9 bootstrap rather than the current one...

It might be nicer to get DeuTex to stop having a dependency on any IWAD at all, but telling to download Freedoom as an alternative wouldn't be bad.

@Voros2
Copy link
Contributor Author

Voros2 commented Jul 17, 2017

Given how the bootstrap IWAD of Freedoom is deleted after each build, I had to link to 0.9's IWAD instead, as an example.

Including a simple doom2.wad bootstrap IWAD would suffice, since Doom 2 PWADs are what most people churn out these days.

As for eliminating IWAD dependency, that certainly would be a good idea, but surely the IWAD dependency was added for a good reason?

@fragglet
Copy link
Contributor

fragglet commented Jul 27, 2017

As I recall, one of the main reasons for the bootstrap WAD is that deutex will include texture information from an existing IWAD. This is necessary when generating a PWAD file with modified textures, but when generating an IWAD it makes little sense. In particular it's problematic for Freedoom because it means that texture information might be automatically included from a copyrighted IWAD file. Maybe add some conditional code that will ignore existing IWADs when generating an IWAD?

@Voros2
Copy link
Contributor Author

Voros2 commented Jul 28, 2017

Since Deutex already has support for recognising IWADs and PWADs via Type, would something like
if(Type==IWAD) { //stuff if IWAD } else { //stuff if PWAD }
be also considered legible?

@fragglet
Copy link
Contributor

No, I meant if the -iwad flag is given on the command line.

@Voros2
Copy link
Contributor Author

Voros2 commented Jul 28, 2017

Well, -iwad does set Type to IWAD instead of the default PWAD, which is basically what I was getting at with that if loop.

You can find the -iwad command in deutex.c.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants