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

Addition: Specify include directories #27

Closed
WildCard65 opened this issue Apr 10, 2015 · 6 comments
Closed

Addition: Specify include directories #27

WildCard65 opened this issue Apr 10, 2015 · 6 comments

Comments

@WildCard65
Copy link

This would benefit a lot of people to be able to tell SPEdit where to look for includes no matter what configuration is selected(Example: main include dirs hold all the stock SM includes, extra includes include things like: /ff2_includes which holds the includes things like FF2 rely on. SPComp supports this as far as I know of.

@JulienKluge
Copy link
Owner

Spcomp supports that? I mean I'll look into the code but i would highly doubt it.
You can specify the includes in the script to tell spcomp where it should search but i dont think that the -i switch would support multiple directorys (there's also no syntax defined for that case).
I'll never the less look in the compiler code when i am home.
The problem is: i could easily implement multiple directories. But as you sad: does spcomp support this?

@WildCard65
Copy link
Author

I looked in SM's AMBuilder file for the compile process of the sm plugins and found 2 -I switches.
Edit: Did a test run and got the following:
spcomp ff2_test.sp -ocompile/ff2_test.smx -ipath_to_my_downloads_folder/scripting/include
ff2_test.sp includes freak_fortress_2.inc which the inc is in path_to_my_downloads_folder/scripting/include
spcomp said it couldn't find morecolors.inc(freak_fortress_2.inc depends on morecolors.inc)

@WildCard65
Copy link
Author

To add onto this, probably be best to only supply 1 spcomp.exe(the 1.7 one) and just pass it an -i to appropriate include path.

@WheteThunger
Copy link

This is certainly a must-have feature and the compiler already supports it with spcomp.exe -ipath1 -ipath2. I have needed to use it in several cases, and one person I work with does this all the time to separate core/external includes from his git repo.

@JulienKluge
Copy link
Owner

If the Compiler supports multiple include directorys, and I'll test that too, this will come..in the future..

JulienKluge added a commit that referenced this issue May 3, 2015
Implement multiple include directories since i tested it and it works
well. Some behaviours had to be rewritten because they werent able to
handle multiple include directories. But now they are.
@JulienKluge
Copy link
Owner

implemented and thanks.

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