-
Notifications
You must be signed in to change notification settings - Fork 35
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
[Refactor] Add paradiseo paths on the all #include directives #40
Comments
Yes, it's an historic remaining. I believe the firsts authors planned to only use The classical organization would be to use relative includes within the framework, and use full paths |
So, an adaptation for relative include paths would help in utilization of the EO module? Or it isn't needed? Another question: What the impact of this modification in relation to original EvolvingObjects project? ParadisEO can modify the original source code? I proppose me to carefully reorganize the include structure using relative |
The EO project have been merged within ParadisEO and is now maintained here. As it's Free Software, anyone can actually modify the (licensed) source code in its own fork. So far, I'm not aware of another EO fork, anyway. For the relative include update, I think one should explain the plan on the mailing-list, because I fear I'm missing something here. Can you subscribe to ParadisEO devel ML and post a description of your proposed refactoring, to get some comments? |
Absolute includes considers the system paths
The includes within source files is in the form:
I believe this form considers the install path of EO, but when ParadisEO was installed, the include path is
*/paradiseo/eo/
.It forces us to use the
-I
flag ong++
command, including the complete path on compile command.Add the paradiseo path improvements these situation.
In this case:
... in all source files.
The text was updated successfully, but these errors were encountered: