Command line options for adding directories to include-path should prepend, not append

[felix winkelmann]

Options for csc, chicken and csi that add drectories to the current include path should prepend the added dirs instead of appending them, so the explicitly given include directories take precedence before previously existing ones.

This is consistent to how C compilers handle include paths.

[felix winkelmann]

After checking the code for csi, chicken and csc, I think this behaviour seems to already the case.

Am I missing something?

[sjamaan]

Could this have been fixed by df49ac575c7e41c1ee5655b7d33b6575925db718?

[sjamaan]

(specifically with the introduction of ##sys#split-path)

[sjamaan]

Or could this have something to do with the fact that the CSC_OPTIONS and CSI_OPTIONS environment variables are appended at the end, even after the initial value of ##sys#include-pathnames (which is the result of (list (chicken-home)))?

[felix winkelmann]

Replying to sjamaan:

Or could this have something to do with the fact that the CSC_OPTIONS and CSI_OPTIONS environment variables are appended at the end, even after the initial value of ##sys#include-pathnames (which is the result of (list (chicken-home)))?

But what would be the "right" behaviour? Currently the options from the env vars are prepended in csc.scm.

[sjamaan]

Replying to felix:

Replying to sjamaan:

Or could this have something to do with the fact that the CSC_OPTIONS and CSI_OPTIONS environment variables are appended at the end, even after the initial value of ##sys#include-pathnames (which is the result of (list (chicken-home)))?

But what would be the "right" behaviour? Currently the options from the env vars are prepended in csc.scm.

I'd expect explicitly supplied options on the command line to supersede options given in the environment, so I suppose prepending the options is correct. However, currently -I will be processed in order which means earlier options are searched first, so any -I in the CSC_OPTIONS will supersede any -I on the command line.

I don't know if there's a "correct" solution for this problem. Reversing the include-path seems weird too, because something like csc -I /foo -I /bar would then cause the compiler to search /bar before /foo, which is definitely not what one would expect.

All in all, I think the current situation is probably fine as it is. So maybe this should just be closed "works for me" and wait until someone comes up with a clearer example of what's wrong with the current situation?

[felix winkelmann]

Note that CSC_OPTIONS and CSI_OPTIONS are _prepended_ and thus superseded by explicit arguments, at least that is how I read the current implementation.

But I think you're right. There doesn't seem to be a correct way.