-
-
Notifications
You must be signed in to change notification settings - Fork 661
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
[hxb] Filter for --hxb output #11490
Comments
Quick idea: allow |
Yeah I wanted to talk about filtering output too, but kept that problem for "later" (which this issue does perfectly too :D) Not sure about that json config part (which wouldn't help with --hxb output either) This would also mean haxe compiler would have to check that file in all requests in case it changes, etc. Not sure we'd really gain much there either (target conditional and such would still be weird in json?) |
... I meant |
I guess it's not really an issue to have a file there then :D But yeah that |
It aligns with |
Would be nice if this could be controlled from macros too somehow (beyond fiddling with the zip file in |
I'd be curious what type of configuration would go into a JSON besides the excluded path names? I would like it to be possible to be controlled by a macro, and have easy access inside an hxml, so perhaps it could use an api similar to |
I was thinking of doing it like this: typedef HxbConfig = {
var archivePath:String;
var exclude:Array<String>; // or something
// more later
}
class Compiler {
extern static public function getHxbConfig():HxbConfig;
extern static public function setHxbConfig(config:HxbConfig):Void;
} Then normal people can use |
What's the purpose of setting the |
This is for export ( |
One thing I've been ignoring so far is that we currently can't really specify what actually goes into a generated .hxb file. For library usage, we typically don't want to include code from std and such, so it would be nice to be able to configure this somehow.
It's probably not super urgent because I'm the kind of guy who will just tell someone to delete things from the generated zip file... but maybe usability could be a little better than that.
The text was updated successfully, but these errors were encountered: