Re: Resource files support (v2)

From: Alexey Pakhunov (lx_uk_at_[hidden])
Date: 2004-06-01 13:31:21

Hi Andre,

> First, rc.jam has:
>
> # Since it's a common practice to write
> # exe hello : hello.cpp hello.rc
> # we change the name of object created from RC file, to
> # avoid conflict with hello.cpp.
> generators.register-standard rc.resource-compile : RC : OBJ(%_res) ;
>
> which was a requirement from someone. It should be trivial to include
> that along with your modifications.

Actually I thought that this is workaround for possible file names conflict:

exe hello : hello.rc hello.c ;

since the first version of rc.jam registered .rc -> .obj transformation:

generators.register-standard rc.resource-compile : RC : OBJ ;

But in my case invoking rc.exe produces .res file, so we get "hello.obj" and
"hello.res" which are processed correctly:

generators.register-standard msvc.compile.rc : RC : RES : <toolset>msvc
;

I guess nobody make assomptions that rc.exe produces *_res.obj files so it is
safe.

BTW: The patch I send was incomplete. :-( I didn't include the following lines:

generators.register-linker msvc.link : OBJ SEARCHED_LIB STATIC_LIB
IMPORT_LIB RES : EXE RSP : <toolset>msvc ;
generators.register-linker msvc.link.dll : OBJ SEARCHED_LIB STATIC_LIB
IMPORT_LIB RES : SHARED_LIB IMPORT_LIB RSP : <toolset>msvc ;
generators.register-composing msvc.archive : OBJ RES : STATIC_LIB RSP :
<toolset>msvc ;

Withouth them bjam won't be able to link resources to resulting dll/exe/lib. The
attached file is corrected version of the patch.

> Second, I believe that if you are running toolset=msvc under cygwin,
> you still want to run rc.exe, not windres.exe. If you are running

Yes, you are right. Now I just call rc in any case.

-- 
Best regards,
Alexey.
 ------=_NextPart_000_00C6_01C4481F.C80E4160 Content-Type: application/octet-stream;
name="msvc.jam.diff"
Content-Transfer-Encoding: quoted-printable
Content-Disposition: attachment;
filename="msvc.jam.diff"
[Attachment content not displayed.] ------=_NextPart_000_00C6_01C4481F.C80E4160--