Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(1345)

Issue 4454049: [google] Normalize version number for google/gcc-4_6 (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
13 years, 2 months ago by Diego Novillo
Modified:
13 years, 2 months ago
Reviewers:
doko, richard.guenther
CC:
Ollie Wild, gcc-patches_gcc.gnu.org
Visibility:
Public.

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+1 line, -1 line) Patch
M gcc/BASE-VER View 1 chunk +1 line, -1 line 0 comments Download

Messages

Total messages: 5
Diego Novillo
Since google/gcc-4_6 follows the 4.6 branch, changes in minor revisions cause unnecessary churn in directory ...
13 years, 2 months ago (2011-05-02 19:53:53 UTC) #1
Ollie Wild
On Mon, May 2, 2011 at 2:53 PM, Diego Novillo <dnovillo@google.com> wrote: > > Since ...
13 years, 2 months ago (2011-05-02 19:57:57 UTC) #2
doko_ubuntu.com
On 05/02/2011 09:53 PM, Diego Novillo wrote: > Since google/gcc-4_6 follows the 4.6 branch, changes ...
13 years, 2 months ago (2011-05-03 22:21:05 UTC) #3
richard.guenther_gmail.com
On Wed, May 4, 2011 at 12:20 AM, Matthias Klose <doko@ubuntu.com> wrote: > On 05/02/2011 ...
13 years, 2 months ago (2011-05-04 09:30:28 UTC) #4
Diego Novillo
13 years, 2 months ago (2011-05-04 11:29:16 UTC) #5
On Tue, May 3, 2011 at 18:20, Matthias Klose <doko@ubuntu.com> wrote:
> On 05/02/2011 09:53 PM, Diego Novillo wrote:
>>
>> Since google/gcc-4_6 follows the 4.6 branch, changes in minor
>> revisions cause unnecessary churn in directory names.
>>
>> Fixed with this.  OK for google/gcc-4_6?
>>
>>        Google ref 4335466.
>>
>>        * BASE-VER: Change to 4.6.x-google.
>>
>> diff --git a/gcc/BASE-VER b/gcc/BASE-VER
>> index 4110f74..33d4edd 100644
>> --- a/gcc/BASE-VER
>> +++ b/gcc/BASE-VER
>> @@ -1 +1 @@
>> -4.6.1-google
>> +4.6.x-google
>
> is this enough? the subminor version number is encoded in more places, e.g.
> C++ headers, Go libraries, jar files.

It work here so far.  All the places where the version number is
encoded in ultimately get it from BASE-VER.  For us, the important
thing is to avoid the churn in installed directory names and such.
What the version string says is not as important.


Diego
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld f62528b