[ECOS] "target install/include/pkgconf is not a valid existing directory"

[ECOS] "target install/include/pkgconf is not a valid existing directory"

[John Gumb]

Folks,

FWIW, I tried building the command line ecosconfig.exe under cygwin on NT
yesterday (i.e. NOT the MSDEV/graphical tools). Initially, I got the error
messsage

"target install/include/pkgconf is not a valid existing directory"

when doing 'ecosconfig tree'. I tracked this down to generate_build_tree in
build.cxx.

For fun, I guessed paths weren't getting set up correctly so I hacked out
the #ifdef _WIN32 and forced the inclusion of the path fixup code in
$ECOS_REPOSITORY\host\tools\configtool\common\common\build.cxx line 548ish.

All works a treat. 

I know this is a gross hack - if I've missed something them pls let me know.

Cheers,

John

$ECOS_REPOSITORY\host\tools\configtool\common\common\build.cxx line 548ish.

// generates the directory structure for the build and install trees
bool generate_build_tree (const CdlConfiguration config, const std::string
build_tree, const std::string install_tree /* = "" */ ) {
/* #ifdef _WIN32 [jag] */
#if 1
	// convert backslash directory separators to forward slashes under
Win32
	const std::string build_dir = replace_char (build_tree, '\\', '/');
	const std::string install_dir = install_tree.empty () ? build_dir +
"/install" : replace_char (install_tree, '\\', '/');
#else
	const std::string build_dir = build_tree;
	const std::string install_dir = install_tree.empty () ? build_dir +
"/install" : install_tree;
#endif

Re: [ECOS] "target install/include/pkgconf is not a valid existing directory "

[Julian Smart]

Thanks, I'll adjust the sources accordingly... we must have assumed that 
_WIN32 would be defined under Cygwin too.

Regards,

Julian

At 12:15 PM 8/23/01 +0100, John Gumb wrote:
>Folks,
>
>FWIW, I tried building the command line ecosconfig.exe under cygwin on NT
>yesterday (i.e. NOT the MSDEV/graphical tools). Initially, I got the error
>messsage
>
>"target install/include/pkgconf is not a valid existing directory"
>
>when doing 'ecosconfig tree'. I tracked this down to generate_build_tree in
>build.cxx.
>
>For fun, I guessed paths weren't getting set up correctly so I hacked out
>the #ifdef _WIN32 and forced the inclusion of the path fixup code in
>$ECOS_REPOSITORY\host\tools\configtool\common\common\build.cxx line 548ish.
>
>All works a treat.
>
>I know this is a gross hack - if I've missed something them pls let me know.
>
>Cheers,
>
>John
>
>$ECOS_REPOSITORY\host\tools\configtool\common\common\build.cxx line 548ish.
>
>// generates the directory structure for the build and install trees
>bool generate_build_tree (const CdlConfiguration config, const std::string
>build_tree, const std::string install_tree /* = "" */ ) {
>/* #ifdef _WIN32 [jag] */
>#if 1
>         // convert backslash directory separators to forward slashes under
>Win32
>         const std::string build_dir = replace_char (build_tree, '\\', '/');
>         const std::string install_dir = install_tree.empty () ? build_dir +
>"/install" : replace_char (install_tree, '\\', '/');
>#else
>         const std::string build_dir = build_tree;
>         const std::string install_dir = install_tree.empty () ? build_dir +
>"/install" : install_tree;
>#endif

--
Red Hat UK Ltd, Unit 200 Rustat House, 62 Clifton Road, Cambridge, UK. CB1 
7EG Tel: +44 (1223) 271063