Bug 104860 – crossdev avr binutils: avr-as doesn't support $ symbol in identifiers

• Product: Gentoo Linux
• Status: VERIFIED FIXED

[Description Sandro Bonazzola (RETIRED) 2005-09-05 00:34:21 UTC]

Using avr-as with TinyOS requires that avr-as could accept $ symbols in
identifiers. In nesc-1.1.2 there was a patched version of as that was installed
with the package itself. In the upcoming nesc-1.2 the patched version of as will
not be included.
Here is some hints for solving the problem:
https://mail.millennium.berkeley.edu/pipermail/tinyos-2.0wg/2005-July/000993.html
https://mail.millennium.berkeley.edu/pipermail/tinyos-2.0wg/2005-July/000994.html
https://mail.millennium.berkeley.edu/pipermail/tinyos-2.0wg/2005-July/000998.html

This bug block newer version of nesc.

Reproducible: Always
Steps to Reproduce:
1. copy tinyos ebuilds in your local overlay
2. emerge tinyos
3. try to compile any application for mica2 (wich use avr)

Actual Results:  
avr-as refuse to compile because there are unknown opcodes (due to $ symbol in
identifier)

Expected Results:  
avr-as should compile code with $ symbols.

[Comment 1 SpanKY 2005-09-05 13:56:45 UTC]

i dont support avr

if you want it fixed, attach a patch to the bug

[Comment 2 Scott Price 2005-09-05 18:53:40 UTC]

Is this the only application of this patch?

What are the $ signs used for?  Are they only part of text identifiers?

If this is the only application then I question why you would patch avr-as
instead of tinyOS.

I also agree.  We do not have time to go looking for patches.  You need to
attach them to the bug report.

Scott  =)

[Comment 3 Sandro Bonazzola (RETIRED) 2005-09-16 14:12:01 UTC]

Created attachment 68618 [details, diff]
reintroducing $ simbol in avr opcode

produced by diff -u -r binutils-2.15 avr-binutils-2.15tinyos
I've not tested it against binutils-2.16.
I've not tested it with any other patch applied by gentoo.
I've planned to give it a try ASAP but I can't say when I'll can do it.

[Comment 4 Sandro Bonazzola (RETIRED) 2005-09-16 14:12:45 UTC]

I hope this patch could be useful.

[Comment 5 SpanKY 2005-09-16 15:01:32 UTC]

ok, that patch is good enough for me to tweak and add, thanks

[Comment 6 SpanKY 2005-09-16 15:13:01 UTC]

added to binutiles-2.16.1, no rev bump :P

[Comment 7 Sandro Bonazzola (RETIRED) 2005-09-18 10:47:34 UTC]

Tested, works fine for me. 

[Comment 8 Sandro Bonazzola (RETIRED) 2005-09-18 10:48:20 UTC]

Thanks.