[Pkg-ace-devel] Bug#346607: Depending on x-dev (was: Re: Bug#346607: Processed: xlibs-dev nmu ace)

[Thomas Girard]

I asked:
>>why was x-dev dependency added to the `Build-Depends:' field ? As
>>`libxt-dev` already depends on it (via e.g. libx11-dev), I don't think
>>it should be marked twice. Am I missing something ?

and Steve replied:
> You use headers from x-dev directly; you shouldn't rely on transitive
> dependencies to pull it in, as those are subject to change.

Thanks for the explanation. I have checked again but I don't believe
your explanation applies here:

  me at mymachine:ACE_wrappers$ find . -name '*.i' -o -name '*.cpp' -o 
-name '*.h' -o name '*.inl' | xargs grep X11
  1 ./ace/FlReactor.cpp:  // First clean up the corresponding X11Input.
  2 ./ace/TkReactor.cpp:  // First clean up the corresponding X11Input.
  3 ./ace/XtReactor.cpp:  // First clean up the corresponding X11Input.
  4 ./ace/XtReactor.h:#include /**/ <X11/Intrinsic.h>
  5 ./tests/XtAthenaReactor_Test.cpp:#include /**/ <X11/Intrinsic.h>
  6 ./tests/XtAthenaReactor_Test.cpp:#include /**/ <X11/Xatom.h>
  7 ./tests/XtAthenaReactor_Test.cpp:#include /**/ <X11/Shell.h>
  8 ./tests/XtAthenaReactor_Test.cpp:#include /**/ <X11/Xaw/Command.h>
  9 ./tests/XtAthenaReactor_Test.cpp:#include /**/ <X11/Xaw/Label.h>
10 ./tests/XtAthenaReactor_Test.cpp:#include /**/ <X11/Xaw/Box.h>
11 ./tests/XtAthenaReactor_Test.cpp:#include /**/ <X11/StringDefs.h>
12 ./tests/XtMotifReactor_Test.cpp:#include /**/ <X11/Intrinsic.h>
13 ./tests/XtMotifReactor_Test.cpp:#include /**/ <X11/Xatom.h>
14 ./tests/XtMotifReactor_Test.cpp:#include /**/ <X11/Shell.h>
15 ./TAO/tao/XtResource_Factory.h:#include /**/ <X11/Intrinsic.h>
16 ./TAO/tao/XtResource_Loader.h:#include /**/ <X11/Intrinsic.h>

 From the above results, only line 4, 15 and 16 are relevant. The other
ones are either not compiled or false positive.

Thanks,

Thomas