[Bug binutils/6714] WindRes 2.18 can't concatenate literal strings anymore?

2009-07-01 Thread cvs-commit at gcc dot gnu dot org

--- Additional Comments From cvs-commit at gcc dot gnu dot org  2009-07-01 
15:52 ---
Subject: Bug 6714

CVSROOT:/cvs/src
Module name:src
Changes by: ni...@sourceware.org2009-07-01 15:52:28

Modified files:
binutils   : ChangeLog rclex.c 

Log message:
PR 6714
* rclex.c (handle_quotes): Do not complain about whitespace
separating quoted strings.
(rclex_string): Ignore whitespace when looking for a new opening
quote following a closing quote.

Patches:
http://sources.redhat.com/cgi-bin/cvsweb.cgi/src/binutils/ChangeLog.diff?cvsroot=src&r1=1.1490&r2=1.1491
http://sources.redhat.com/cgi-bin/cvsweb.cgi/src/binutils/rclex.c.diff?cvsroot=src&r1=1.6&r2=1.7



-- 


http://sourceware.org/bugzilla/show_bug.cgi?id=6714

--- You are receiving this mail because: ---
You are on the CC list for the bug, or are watching someone who is.


___
bug-binutils mailing list
bug-binutils@gnu.org
http://lists.gnu.org/mailman/listinfo/bug-binutils


[Bug binutils/6714] WindRes 2.18 can't concatenate literal strings anymore?

2009-07-01 Thread nickc at redhat dot com

--- Additional Comments From nickc at redhat dot com  2009-07-01 15:52 
---
Hi Carl,

  Thank you for submitting a test case, and sorry for taking so long to reply to
your posting.

  You are correct, windres was not concatenating quoted strings as it ought to
have been doing.  I am going to apply the uploaded patch to fix this problems. 
it works with the test case you provided but if you find any more circumstances
under which concatenation does not work, please reopen this PR and submit a new
test case.

Cheers
  Nick

binutils/ChangeLog
PR 6714
* rclex.c (handle_quotes): Do not complain about whitespace
separating quoted strings.
(rclex_string): Ignore whitespace when looking for a new opening
quote following a closing quote.


-- 
   What|Removed |Added

 Status|NEW |RESOLVED
 Resolution||FIXED


http://sourceware.org/bugzilla/show_bug.cgi?id=6714

--- You are receiving this mail because: ---
You are on the CC list for the bug, or are watching someone who is.


___
bug-binutils mailing list
bug-binutils@gnu.org
http://lists.gnu.org/mailman/listinfo/bug-binutils


[Bug binutils/6714] WindRes 2.18 can't concatenate literal strings anymore?

2009-07-01 Thread nickc at redhat dot com

--- Additional Comments From nickc at redhat dot com  2009-07-01 15:48 
---
Created an attachment (id=4033)
 --> (http://sourceware.org/bugzilla/attachment.cgi?id=4033&action=view)
Join quoted strings separated by whitespace.


-- 


http://sourceware.org/bugzilla/show_bug.cgi?id=6714

--- You are receiving this mail because: ---
You are on the CC list for the bug, or are watching someone who is.


___
bug-binutils mailing list
bug-binutils@gnu.org
http://lists.gnu.org/mailman/listinfo/bug-binutils


[Bug binutils/6714] WindRes 2.18 can't concatenate literal strings anymore?

2009-05-13 Thread stu at nosnilmot dot com

--- Additional Comments From stu at nosnilmot dot com  2009-05-13 19:34 
---
Here's a test case:

$ cat stringconcattest.rc 
#define PIXMAPSDIR "pixmaps/"

TRAY_ICON_TEST  ICON PIXMAPSDIR "testicon.ico"

$ ls -l pixmaps/testicon.ico 
-rw-rw-r--. 1 stu stu 318 2009-05-13 20:30 pixmaps/testicon.ico

$ i686-pc-mingw32-windres -i stringconcattest.rc -o stringconcattest.o
i686-pc-mingw32-windres: icon file `pixmaps/' does not contain icon data

$ i686-pc-mingw32-windres --version
GNU windres (GNU Binutils) 2.19.1


-- 
   What|Removed |Added

 CC||stu at nosnilmot dot com


http://sourceware.org/bugzilla/show_bug.cgi?id=6714

--- You are receiving this mail because: ---
You are on the CC list for the bug, or are watching someone who is.


___
bug-binutils mailing list
bug-binutils@gnu.org
http://lists.gnu.org/mailman/listinfo/bug-binutils


[Bug binutils/6714] WindRes 2.18 can't concatenate literal strings anymore?

2009-02-11 Thread nickc at redhat dot com

--- Additional Comments From nickc at redhat dot com  2009-02-11 14:53 
---
Hi Craig,

  I think that you are going to have to post another test case to demonstrate
the problem that you are having.  I tried the following:

  % cat fred.inc
  #define VALUE_AS_TXT "15"

  % cat fred.rc
  #include "fred.inc"

  101 DIALOG DISCARDABLE  0, 0, 186, 95
  BEGIN
 LTEXT   "Value: " VALUE_AS_TXT, -1, 7, 238, 100,  16
 LTEXT   "Value: " "16", -1, 7, 238, 100,  16
 LTEXT   "Value: 17",-1, 7, 238, 100,  16
  END

And got this result:

  % windres fred.rc
  LANGUAGE 9, 1

  101 DIALOG MOVEABLE PURE DISCARDABLE 0, 0, 186, 95
  STYLE 0x8088
  BEGIN
LTEXT "Value: 15", -1, 7, 238, 100, 16, 0x5002
LTEXT "Value: 16", -1, 7, 238, 100, 16, 0x5002
LTEXT "Value: 17", -1, 7, 238, 100, 16, 0x5002
  END


So string literal concatenation is working.

Cheers
  Nick

-- 


http://sourceware.org/bugzilla/show_bug.cgi?id=6714

--- You are receiving this mail because: ---
You are on the CC list for the bug, or are watching someone who is.


___
bug-binutils mailing list
bug-binutils@gnu.org
http://lists.gnu.org/mailman/listinfo/bug-binutils


[Bug binutils/6714] WindRes 2.18 can't concatenate literal strings anymore?

2009-02-11 Thread nickc at redhat dot com

--- Additional Comments From nickc at redhat dot com  2009-02-11 14:40 
---
Doh! Please ignore suggestion 2.  I had completely blanked on this already being
in bugzilla.


-- 


http://sourceware.org/bugzilla/show_bug.cgi?id=6714

--- You are receiving this mail because: ---
You are on the CC list for the bug, or are watching someone who is.


___
bug-binutils mailing list
bug-binutils@gnu.org
http://lists.gnu.org/mailman/listinfo/bug-binutils


[Bug binutils/6714] WindRes 2.18 can't concatenate literal strings anymore?

2009-02-11 Thread nickc at redhat dot com

--- Additional Comments From nickc at redhat dot com  2009-02-11 14:38 
---
Subject: Re:  WindRes 2.18 can't concatenate literal strings
 anymore?

Hi Craig,

> I'm using 2.19 and I still get the same error of :
> 
> /opt/mingw32/bin/i586-pc-mingw32-windres: icon file `pixmaps/' does not 
> contain
> icon data
> 
> Is there a temp fix I can apply to get this working..?

1. Create a patch to fix windres so that it will concatenate string 
literals.

or:

2. Submit a bug report to http://sourceware.org/bugzilla/ describing the 
problem and including a small test case to reproduce the problem.

or:

3. Edit your resource file and perform the string literal concatenation 
yourself.

or:

4. Create a tool in your favourite programming language to pre-process 
the resources files for you.

Cheers
   Nick



-- 


http://sourceware.org/bugzilla/show_bug.cgi?id=6714

--- You are receiving this mail because: ---
You are on the CC list for the bug, or are watching someone who is.


___
bug-binutils mailing list
bug-binutils@gnu.org
http://lists.gnu.org/mailman/listinfo/bug-binutils


Re: [Bug binutils/6714] WindRes 2.18 can't concatenate literal strings anymore?

2009-02-11 Thread Nick Clifton

Hi Craig,


I'm using 2.19 and I still get the same error of :

/opt/mingw32/bin/i586-pc-mingw32-windres: icon file `pixmaps/' does not contain
icon data

Is there a temp fix I can apply to get this working..?


1. Create a patch to fix windres so that it will concatenate string 
literals.


or:

2. Submit a bug report to http://sourceware.org/bugzilla/ describing the 
problem and including a small test case to reproduce the problem.


or:

3. Edit your resource file and perform the string literal concatenation 
yourself.


or:

4. Create a tool in your favourite programming language to pre-process 
the resources files for you.


Cheers
  Nick



___
bug-binutils mailing list
bug-binutils@gnu.org
http://lists.gnu.org/mailman/listinfo/bug-binutils


[Bug binutils/6714] WindRes 2.18 can't concatenate literal strings anymore?

2009-02-06 Thread craigwharding at gmail dot com

--- Additional Comments From craigwharding at gmail dot com  2009-02-07 
00:35 ---
(In reply to comment #1)
> Subject: Re:  New: WindRes 2.18 can't concatenate 
>  literalstrings anymore?
> 
> c dot colijn at twologs dot com wrote:
> 
> > windres.exe: C:\\Source\\TIMETR~1.RC:137: syntax error
> > windres.exe: preprocessing failed.
> 
> This was fixed some time ago in CVS, please try a more recent version.
> 

I was searching for my error with windres and found this
(http://www.dronecode.org.uk/daybook/2008/08/pidgin-monotone-crosscompile-for-win32.html),.
The fourth point describes my problem. That link led me here. 

I'm using 2.19 and I still get the same error of :

/opt/mingw32/bin/i586-pc-mingw32-windres: icon file `pixmaps/' does not contain
icon data

Is there a temp fix I can apply to get this working..?

-- 
   What|Removed |Added

 CC||craigwharding at gmail dot
   ||com


http://sourceware.org/bugzilla/show_bug.cgi?id=6714

--- You are receiving this mail because: ---
You are on the CC list for the bug, or are watching someone who is.


___
bug-binutils mailing list
bug-binutils@gnu.org
http://lists.gnu.org/mailman/listinfo/bug-binutils


[Bug binutils/6714] WindRes 2.18 can't concatenate literal strings anymore?

2008-07-01 Thread brian at dessent dot net

--- Additional Comments From brian at dessent dot net  2008-07-01 17:12 
---
Subject: Re:  New: WindRes 2.18 can't concatenate 
 literalstrings anymore?

c dot colijn at twologs dot com wrote:

> windres.exe: C:\\Source\\TIMETR~1.RC:137: syntax error
> windres.exe: preprocessing failed.

This was fixed some time ago in CVS, please try a more recent version.


-- 


http://sourceware.org/bugzilla/show_bug.cgi?id=6714

--- You are receiving this mail because: ---
You are on the CC list for the bug, or are watching someone who is.


___
bug-binutils mailing list
bug-binutils@gnu.org
http://lists.gnu.org/mailman/listinfo/bug-binutils


[Bug binutils/6714] WindRes 2.18 can't concatenate literal strings anymore?

2008-07-01 Thread c dot colijn at twologs dot com


-- 
   What|Removed |Added

 GCC target triplet||Windows XP SP3


http://sourceware.org/bugzilla/show_bug.cgi?id=6714

--- You are receiving this mail because: ---
You are on the CC list for the bug, or are watching someone who is.


___
bug-binutils mailing list
bug-binutils@gnu.org
http://lists.gnu.org/mailman/listinfo/bug-binutils