579829 - Use "macosx64" instead of "macosx-snow" in buildbot-configs

Status: RESOLVED FIXED

(Release Engineering :: General, defect, P2)

Description

[Rail Aliiev [:rail]]

Attachment: Use "macosx64" instead of "macosx-snow" in buildbot-configs

Using "macosx-snow" for slave names introduces duplication of code (see bug 578749 comment #10). It will be better to use the same name as we use for the platforms.

Comment 1

[[e-mail] (:bhearsum)]

I'd argue that "macosx64" is incorrect, actually. The uname of these machines is "i386", not "x86_64".

Comment 2

[Mike Taylor [:bear]]

(In reply to comment #1)
> I'd argue that "macosx64" is incorrect, actually. The uname of these machines
> is "i386", not "x86_64".

The uname is i386 true, but that doesn't convey that they can be used for 64bit builds because of the OS installed - IMO

Comment 3

[[e-mail] (:bhearsum)]

IMHO slave names should represent the operating system, not what their capable of doing.

Comment 4

[Mike Taylor [:bear]]

(In reply to comment #3)
> IMHO slave names should represent the operating system, not what their capable
> of doing.

I agree with that, but I thought that Rail's change was just to bring the osx64 dictionary key names in line with the others and doesn't impact slave names.  the pattern until "snow" had been foo32 and foo64

sorry if my earlier reply was confusing/misleading - I should have stayed out of the bug queue until later when I could properly focus on it

Comment 5

[[e-mail] (:bhearsum)]

Sorry, when I say "slave name" I'm speaking of both the hostnames as well as the BuildSlave "slavenames".

32 and 64 are appropriate for the linuxes, because they are indeed 32 and 64-bit host platforms. Same thing for Windows. I'm aware that we used "mac64" in a couple of other places but IMHO we should fix those rather than this.

Comment 6

[Mike Taylor [:bear]]

then i'll change my r+ to a r- and ask Rail if he doesn't mind doing that

+1 ?

Comment 7

[Mike Taylor [:bear]]

Comment on attachment 458265 [details] [diff] [review]
Use "macosx64" instead of "macosx-snow" in buildbot-configs

see thread above for reason why I changed :)

Comment 8

[[e-mail] (:bhearsum)]

We talked about this a bit on the phone. My feelings on this matter aren't very strong, you can land this as far as I'm concerned.

Comment 9

[Rail Aliiev [:rail]]

I'd be happy if we land it, because we need it for mac64 l10n nightlies.

Comment 10

[Rail Aliiev [:rail]]

Attachment: Use "macosx64" instead of "macosx-snow" in buildbot-configs

Refreshed patch.

Comment 11

[Rail Aliiev [:rail]]

Comment on attachment 459134 [details] [diff] [review]
Use "macosx64" instead of "macosx-snow" in buildbot-configs

http://hg.mozilla.org/build/buildbot-configs/rev/04ed26127a9b