Pages: [1]   Go Down
Author Topic: SVN keywords not working  (Read 633 times)
0 Members and 1 Guest are viewing this topic.
Dallas, TX USA
Offline Offline
Faraday Member
**
Karma: 70
Posts: 2738
View Profile
 Bigger Bigger  Smaller Smaller  Reset Reset

The SVN keywords for files in the build tree (at least on the files in hardware/arduino/cores/arduino) are not working properly.
Many files have old stale SVN keywords in them.
many are 3+ years old...  :o
The files don't seem to have the proper the keywords property
value to enable keyword exapansion.

--- bill
Logged

Cape Town, South Africa
Offline Offline
Full Member
***
Karma: 0
Posts: 177
Arduino rocks
View Profile
 Bigger Bigger  Smaller Smaller  Reset Reset

There is no keywords file in that directory.

http://code.google.com/p/arduino/source/browse/#svn/trunk/hardware/arduino/cores/arduino

Perhaps it has been deleted, and you are 'showing deleted files' ?
Logged

Dallas, TX USA
Offline Offline
Faraday Member
**
Karma: 70
Posts: 2738
View Profile
 Bigger Bigger  Smaller Smaller  Reset Reset

Huh?

I'm talking about SVN keywords.
Things like $Id$ etc... that are embedded in the source code comments.
SVN expands these on checkout and maintains a list of properties
of which keywords that are to be expanded on a per file basis.
For example, take a look at the file wiring_digital.c
http://code.google.com/p/arduino/source/browse/trunk/hardware/arduino/cores/arduino/wiring_digital.c
line 22.

This file was modified just days ago and is at revision 977 yet the SVN Id keyword in the source code is from 3 years ago at revision 248
because keyword expansion is not properly enabled.
SVN is not like SCCS/RCS/CVS that have keyword expansion
permanently enabled.

Having out of date revision information is worse than having no revision information at all.

The SVN properties need to be updated to allow the SVN keywords to be expanded properly.
There are simple commands to enable keyword expansion
it only has to be done once.
Then the keyword expansion works again, and you can forget about it.

This can only be corrected by one of the arduino project owners.

I'll go open an issue on the issues list so this doesn't get lost.

NOTE:
Link to issue on issues list:
http://code.google.com/p/arduino/issues/detail?id=277


--- bill
« Last Edit: June 21, 2010, 01:38:44 pm by bperrybap » Logged

Cape Town, South Africa
Offline Offline
Full Member
***
Karma: 0
Posts: 177
Arduino rocks
View Profile
 Bigger Bigger  Smaller Smaller  Reset Reset

Ahh, got you. Didn't know that  smiley
Logged

Pages: [1]   Go Up
Jump to: