Remove unused PACKAGE_FILE flag?

RESOLVED FIXED in mozilla1.9.1b2

Status

()

Core
Build Config
RESOLVED FIXED
9 years ago
8 years ago

People

(Reporter: WeirdAl, Assigned: philor)

Tracking

Trunk
mozilla1.9.1b2
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

(URL)

Attachments

(4 attachments)

(Reporter)

Description

9 years ago
It looks like PACKAGE_FILE, according to mxr, is an unused Makefile flag.  Should it and its corresponding .pkg files be removed?

Comment 1

9 years ago
Yeah, they were a half-done attempt at distributing the packaging manifests that never got done and are now completely out of date. Remove them.
(Assignee)

Comment 2

9 years ago
Created attachment 345432 [details] [diff] [review]
mailnews/ fix v.1 [checked in]

This wasn't even entertaining to create, much less to review. I highly recommend grep and find, rather than actually looking at it, as a review technique.
Assignee: nobody → philringnalda
Status: NEW → ASSIGNED
Attachment #345432 - Flags: superreview?(bienvenu)
Attachment #345432 - Flags: review?(bienvenu)
(Assignee)

Updated

9 years ago
Attachment #345432 - Attachment description: Fix v.1 → mailnews/ fix v.1
(Assignee)

Comment 3

9 years ago
Created attachment 345446 [details] [diff] [review]
mozilla-central fix v. 1 [checked in]
Attachment #345446 - Flags: review?(benjamin)
(Assignee)

Comment 4

9 years ago
Created attachment 345453 [details] [diff] [review]
directory/ fix, v. 1 [checked in]

Pretty sure I successfully checked out trunk-directory here...
Attachment #345453 - Flags: review?(bugzilla)

Updated

9 years ago
Attachment #345432 - Flags: superreview?(bienvenu)
Attachment #345432 - Flags: superreview+
Attachment #345432 - Flags: review?(bienvenu)
Attachment #345432 - Flags: review+
Comment on attachment 345453 [details] [diff] [review]
directory/ fix, v. 1 [checked in]

r=me if you apply this to hg only. These files are in hg now.
Attachment #345453 - Flags: review?(bugzilla) → review+
(Assignee)

Comment 6

9 years ago
Comment on attachment 345432 [details] [diff] [review]
mailnews/ fix v.1 [checked in]

http://hg.mozilla.org/comm-central/rev/114d5b34fc31
Attachment #345432 - Attachment description: mailnews/ fix v.1 → mailnews/ fix v.1 [checked in]
(Assignee)

Comment 7

9 years ago
Comment on attachment 345453 [details] [diff] [review]
directory/ fix, v. 1 [checked in]

http://hg.mozilla.org/comm-central/rev/5c72e416c07c
Attachment #345453 - Attachment description: directory/ fix, v. 1 → directory/ fix, v. 1 [checked in]
(Assignee)

Comment 8

9 years ago
Created attachment 345835 [details] [diff] [review]
mailnews/ followup for PACKAGE_VARS [checked in]

It wouldn't be nearly as embarrassing that I apparently didn't yet know that PACKAGE_VARS was part of the same thing, if not for the way I noticed *after* I'd already checked in a patch that didn't remove PACKAGE_VARS with a commit message saying that it did.
Attachment #345835 - Flags: superreview?(bienvenu)
Attachment #345835 - Flags: review?(bienvenu)

Comment 9

9 years ago
Comment on attachment 345835 [details] [diff] [review]
mailnews/ followup for PACKAGE_VARS [checked in]

thx for the patch, Phil!
Attachment #345835 - Flags: superreview?(bienvenu)
Attachment #345835 - Flags: superreview+
Attachment #345835 - Flags: review?(bienvenu)
Attachment #345835 - Flags: review+
(Assignee)

Comment 10

9 years ago
Comment on attachment 345835 [details] [diff] [review]
mailnews/ followup for PACKAGE_VARS [checked in]

http://hg.mozilla.org/comm-central/rev/326265c9d9b8
Attachment #345835 - Attachment description: mailnews/ followup for PACKAGE_VARS → mailnews/ followup for PACKAGE_VARS [checked in]

Comment 11

9 years ago
Comment on attachment 345446 [details] [diff] [review]
mozilla-central fix v. 1 [checked in]

I didn't read the entire patch, just glanced at it and I trust it to be correct...
Attachment #345446 - Flags: review?(benjamin) → review+
Blocks: 462438
(Assignee)

Comment 12

9 years ago
Comment on attachment 345446 [details] [diff] [review]
mozilla-central fix v. 1 [checked in]

http://hg.mozilla.org/mozilla-central/rev/d78c3cd8c9e0
Attachment #345446 - Attachment description: mozilla-central fix v. 1 → mozilla-central fix v. 1 [checked in]
(Assignee)

Updated

9 years ago
Status: ASSIGNED → RESOLVED
Last Resolved: 9 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla1.9.1b2
Version: unspecified → Trunk
You need to log in before you can comment on or make changes to this bug.