public inbox for gcc-patches@gcc.gnu.org
 help / color / mirror / Atom feed
* [google] Normalize version number for google/gcc-4_6 (issue4454049)
@ 2011-05-02 19:54 Diego Novillo
  2011-05-02 19:58 ` Ollie Wild
  2011-05-03 22:34 ` Matthias Klose
  0 siblings, 2 replies; 5+ messages in thread
From: Diego Novillo @ 2011-05-02 19:54 UTC (permalink / raw)
  To: reply, aaw, gcc-patches


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

--
This patch is available for review at http://codereview.appspot.com/4454049

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [google] Normalize version number for google/gcc-4_6 (issue4454049)
  2011-05-02 19:54 [google] Normalize version number for google/gcc-4_6 (issue4454049) Diego Novillo
@ 2011-05-02 19:58 ` Ollie Wild
  2011-05-03 22:34 ` Matthias Klose
  1 sibling, 0 replies; 5+ messages in thread
From: Ollie Wild @ 2011-05-02 19:58 UTC (permalink / raw)
  To: Diego Novillo; +Cc: reply, gcc-patches

On Mon, May 2, 2011 at 2:53 PM, Diego Novillo <dnovillo@google.com> 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?

Yes, okay for google/gcc-4_6.

Thanks,
Ollie

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [google] Normalize version number for google/gcc-4_6 (issue4454049)
  2011-05-02 19:54 [google] Normalize version number for google/gcc-4_6 (issue4454049) Diego Novillo
  2011-05-02 19:58 ` Ollie Wild
@ 2011-05-03 22:34 ` Matthias Klose
  2011-05-04  9:31   ` Richard Guenther
  2011-05-04 11:44   ` Diego Novillo
  1 sibling, 2 replies; 5+ messages in thread
From: Matthias Klose @ 2011-05-03 22:34 UTC (permalink / raw)
  To: Diego Novillo; +Cc: reply, aaw, gcc-patches

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.  For the Debian/Ubuntu packaging I'm just 
using symlinks from 4.6.x to 4.6 to avoid this kind of dependency.  Now that the 
-V option isn't supported anymore, maybe this could be addressed with something 
like a new configure option --version-alias=<value>, similar to the target alias.

   Matthias

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [google] Normalize version number for google/gcc-4_6 (issue4454049)
  2011-05-03 22:34 ` Matthias Klose
@ 2011-05-04  9:31   ` Richard Guenther
  2011-05-04 11:44   ` Diego Novillo
  1 sibling, 0 replies; 5+ messages in thread
From: Richard Guenther @ 2011-05-04  9:31 UTC (permalink / raw)
  To: Matthias Klose; +Cc: Diego Novillo, reply, aaw, gcc-patches

On Wed, May 4, 2011 at 12:20 AM, 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.  For the Debian/Ubuntu packaging I'm
> just using symlinks from 4.6.x to 4.6 to avoid this kind of dependency.  Now
> that the -V option isn't supported anymore, maybe this could be addressed
> with something like a new configure option --version-alias=<value>, similar
> to the target alias.

The SUSE packages introduce gcc/FULL-VER as copy of BASE-VER and
strip the patch-level version from BASE-VER.  Some (but not many)
makefiles need to be adjusted to use FULL-VER but then everything
else just falls out nicely (and you lose the ability to install 4.6.0 and
4.6.1 in parallel, of course).  For some weird marketing reasons we
also drop "prerelease" from the version string and drop the patch-level
version down to the last release.

Richard.

>  Matthias
>

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [google] Normalize version number for google/gcc-4_6 (issue4454049)
  2011-05-03 22:34 ` Matthias Klose
  2011-05-04  9:31   ` Richard Guenther
@ 2011-05-04 11:44   ` Diego Novillo
  1 sibling, 0 replies; 5+ messages in thread
From: Diego Novillo @ 2011-05-04 11:44 UTC (permalink / raw)
  To: Matthias Klose; +Cc: reply, aaw, gcc-patches

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

^ permalink raw reply	[flat|nested] 5+ messages in thread

end of thread, other threads:[~2011-05-04 11:29 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2011-05-02 19:54 [google] Normalize version number for google/gcc-4_6 (issue4454049) Diego Novillo
2011-05-02 19:58 ` Ollie Wild
2011-05-03 22:34 ` Matthias Klose
2011-05-04  9:31   ` Richard Guenther
2011-05-04 11:44   ` Diego Novillo

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).