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

Add :xml subPackage for dlangui to use #54

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

rikkimax
Copy link

Grim isn't too happy that undead isn't building with GDC, this extracts it so it can be used by dlangui until a better solution is found.

@rikkimax
Copy link
Author

I should mention that I didn't test this... but ugh it should be ok

@anon17
Copy link

anon17 commented Jun 9, 2023

AIU undead.xml compiles with gdc just fine (it depends only on phobos), it's undead.stream that doesn't compile, and dub compiles all files in a package. The package won't compile until all files in the package compile.

@ibuclaw
Copy link
Member

ibuclaw commented Jun 10, 2023

AIU undead.xml compiles with gdc just fine (it depends only on phobos), it's undead.stream that doesn't compile,

Correct, as that old module relies on the discouraged use of RTTI based varargs.

@schveiguy
Copy link
Member

Once my PR is merged, this should be closed. We could probably close it now.

@schveiguy
Copy link
Member

I think this can be closed now.

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.

None yet

4 participants