380895 - Clean up missing or unnecessary files in packages-static

Status: RESOLVED FIXED

(Firefox Build System :: General, defect)

Description

[Phil Ringnalda (:philor)]

cf says the spew of "Warning: package error or possible missing or unnecessary file:" is bad enough to be annoying in the release process (and is certainly annoying when you're looking at a Tinderbox "Brief log" trying to find something real in the midst of 87 meaningless lines).

Comment 1

[Phil Ringnalda (:philor)]

Attachment: Trunk v.1

Removed things:

-bin\components\helperAppDlg.xpt
   actually found in exthandler.xpt

-bin\components\intlcmpt.xpt
   removed in bug 352180

-bin\components\jsurl.xpt
   removed in bug 277371

-bin\components\necko_jar.xpt
   removed by bug 248044, re-added by aviary-landing

-bin\components\prefmigr.xpt
   only built ifndef MOZ_SINGLE_PROFILE

-bin\components\related.xpt
-bin\components\search.xpt
   only built for MOZ_SUITE

-bin\components\util.xpt
   created by landing of packaging branch, 2004-01-06 17:22, disappeared when nsITimeRecorder.idl was removed 2004-01-07 14:30

-bin\res\platform-forms.css
   removed by bug 272151

-bin\gksvggdiplus.dll
   --enable-svg-renderer=gdiplus isn't supported on the trunk

-bin\res\svg.properties
   removed from /res/ by bug 296169

-bin\chrome\pipnss.jar
   only contains a contents.rdf we don't build

Missing things we have to live with:

bin/xul.dll
  only built with --enable-libxul
bin/components/bookmarks.xpt
bin/components/nsBookmarkTransactionManager.js
  only built for non-places-bookmarks
bin/components/ucnative.xpt
  only built with --enable-native-uconv
bin/defaults/existing-profile-defaults.js
  used in the confusing changing of defaults process

Things I'm not sure about, so I left them:

bin/components/wallet.xpt
bin/components/walleteditor.xpt
bin/components/walletpreview.xpt
bin/components/signonviewer.xpt
  you certainly can build Fx with wallet, most likely with extensions=all, dunno if it would work, or how to tell

Comment 2

[Phil Ringnalda (:philor)]

browser/installer/windows/packages-static 1.111

Comment 3

[Phil Ringnalda (:philor)]

Attachment: Branch v.1

1.8 branch version, same reasons, just not quite as many (and, sadly, not including places.xpt, since nobody got around to erroring out of configure for --enable-places).

Comment 4

[Daniel Veditz [:dveditz]]

Comment on attachment 266167 [details] [diff] [review]
Branch v.1

approved for 1.8.1.5, a=dveditz for release-drivers

Comment 5

[Phil Ringnalda (:philor)]

browser/installer/windows/packages-static 1.53.2.39