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

Pass additional options to the compiler (-- -l). #249

Open
Y-Less opened this issue Aug 28, 2018 · 4 comments
Open

Pass additional options to the compiler (-- -l). #249

Y-Less opened this issue Aug 28, 2018 · 4 comments

Comments

@Y-Less
Copy link
Contributor

Y-Less commented Aug 28, 2018

I have a lot of configurations defined in pawn.json. Sometimes I want to see why one isn't working. I could add "-l" to the compiler options, but might forget to remove it. I could add a whole new set of build configurations that include "-l", but then I'll have to do the same for "-a" and any other configuration I want (which would take me up to 384 build profiles). It would be nice if, just for testing, I could just give extra options on the command line using the common -- syntax to deliniate. (Un)fortunately the server doesn't take any arguments, so this can't be ambiguous.

@Y-Less
Copy link
Contributor Author

Y-Less commented Aug 28, 2018

Actually, it isn't QUITE that many builds since -l doesn't need all the -O/-d variants, but still a lot.

@Southclaws
Copy link
Owner

I considered something like this in #49 - basically a way to pass build params but also provide more human-readable ones with full names (like -l for short and --listing as the full name).

Should be a fairly easy one to implement, hopefully it will make it into the next release.

@ADRFranklin
Copy link
Collaborator

@Southclaws I noticed you closed issue #49 is that to say that human friendly wasn't a good idea?
@Y-Less

So would this be a options inside the config or options pass to sampctl as a flag, or both? I was in favour of adding human friendly like options to the config.

@Southclaws
Copy link
Owner

Both, I think the ability to add these at build time quickly without editing the config is very useful.

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

3 participants