636215 - browser/locales/Makefile.in cleanup, remove prepare-repackages and prepare-upload-*

Status: RESOLVED FIXED

(Firefox Build System :: General, defect)

Description

[Justin Wood (:Callek)]

Attachment: just clean them up.

prepare-repackages-% and prepare-upload-% are currently unused in browser/locales/Makefile.in

None of our release engineering scripts, nor any existing build rules use them. Just get rid of them, and make life easier to understand.

Comment 1

[Armen [:armenzg]]

Comment on attachment 514547 [details] [diff] [review]
just clean them up.

wow no wonder I barely remember these targets.
They are from back in 2008.
http://hg.mozilla.org/mozilla-central/rev/ccce859a94aa

As we looked around they are not in used anymore.

I see APP_VERSION being used on toolkit files but I can't tell if it is the same one as you are removing.
http://mxr.mozilla.org/mozilla-central/search?string=%25APP_VERSION%25&find=toolkit&findi=&filter=^[^\0]*%24&hitlimit=&tree=mozilla-central

Axel, do you know?

Comment 2

[Justin Wood (:Callek)]

(In reply to comment #1)
> I see APP_VERSION being used on toolkit files but I can't tell if it is the
> same one as you are removing.
> http://mxr.mozilla.org/mozilla-central/search?string=%25APP_VERSION%25&find=toolkit&findi=&filter=^[^\0]*%24&hitlimit=&tree=mozilla-central
> 
> Axel, do you know?

the test/ ones don't expand anything from a makefile, and its app-code that expands them.

The XPIProvider.jsm and plugin*Datasource.js use regex to expand these based on variables local to their respective scopes. None of the makefiles included uses this var as defined here. (fwiw Thunderbird also includes l10n.mk and does not define APP_VERSION in its locales/Makefile.in)

Comment 3

[Justin Wood (:Callek)]

Attachment: just clean them up. v2

Per a chat with armen in IRC, we're keeping the APP_VERSION (with my XXX Comment) just to be extra safe in this end-game. The rest is perfectly safe, and simply unused cleanup.

For added safety once this lands I'll verify with a few repacks from the following nightly, and/or get one or two manually kicked immediately after I land this.

Comment 4

[Justin Wood (:Callek)]

(In reply to comment #3)
> Created attachment 514605 [details] [diff] [review]
> just clean them up. v2

O yea, that r+ is r=armenzg via IRC

Comment 5

[Benjamin Smedberg]

Comment on attachment 514605 [details] [diff] [review]
just clean them up. v2

I can't see why we'd take this now, please renominate if you really need this for some reason that isn't in the bug.

Comment 6

[Justin Wood (:Callek)]

http://hg.mozilla.org/projects/build-system/rev/87c47b419223

Comment 7

[Kyle Huey (Exited; not receiving bugmail, old account, do not use)]

http://hg.mozilla.org/mozilla-central/rev/87c47b419223

Comment 8

[Axel Hecht [:Pike]]

FTR, if you're asking questions to some Axel guy, it might make sense to CC one.