Ticket #189 (closed defect: fixed)

Opened 4 years ago

Last modified 4 years ago

DSSS should put dsss's -I and -S dirs after user's include dirs

Reported by: baxissimo Assigned to: Gregor
Priority: major Milestone:
Component: DSSS Version:
Keywords: Cc:

Description

Currently when dsss calls rebuild, it puts DSSS's include and lib dirs first, before user-specified include and lib directories. This means you always get the installed versions of files even if you explicitly give a different directory to use in your dsss.conf or on the command line.

That means you have no way to override the dsss versions besides to uninstall the dsss versions. But maybe in another project you're relying on the installed versions in most of your projects.

Installed files should never take precedence over files in the include and lib directories manually specified by the user.

Change History

04/12/08 05:49:09 changed by baxissimo

Just to be explicit, here's an example. Currently when I do dsss build on one project I get 

+ f:\usr\pkg\d\dsss\bin\rebuild.exe -Idsss_imports\ -I. -S.\ -If:\usr\pkg\d\dsss\include\d -Sf:\usr\pkg\d\dsss\lib\ -full  -If:\usr\pkg\d\dsss\include\d -Sf:\usr\pkg\d\dsss\lib  -oqdsss_objs\D -I../OpenMeshD/OpenMeshD -I../common -I../ext -g -debug tess_main.d -oftess

I believe that should be changed to this instead: 

+ f:\usr\pkg\d\dsss\bin\rebuild.exe -full  -I../OpenMeshD/OpenMeshD -I../common -I../ext -Idsss_imports\ -I. -S.\ -If:\usr\pkg\d\dsss\include\d -Sf:\usr\pkg\d\dsss\lib\ -oqdsss_objs\D  -g -debug tess_main.d -oftess

Actually you can see in the first one that the DSSS dirs are actually getting added twice currently. Also, I think the automatically added -I. and -S. should go after the user specified flags as well. If I want to force -I. to be the first thing searched I can add it. But I can't un-add something that's implicitly put there by DSSS.

04/12/08 17:38:19 changed by Gregor

  • status changed from new to closed.
  • resolution set to fixed.

I agree. Fixed in revision 894 (which will be 0.76)