Closed Bug 632682 Opened 14 years ago Closed 14 years ago

removed-files.in needs to indicate that a directory is actually a directory.

Categories

(SeaMonkey :: Build Config, defect)

defect
Not set
blocker

Tracking

(blocking-seamonkey2.1 b2+)

RESOLVED FIXED
seamonkey2.1b2
Tracking Status
blocking-seamonkey2.1 --- b2+

People

(Reporter: Callek, Assigned: ewong)

References

Details

Attachments

(1 file)

Ok, build1 of SeaMonkey 2.1beta2 encountered problems on the update generation [verification].

Basically our partial and complete uploads were failing to remove directories, which the update manifests should have properly ignored, this occurred [after investigation] due to not identifying these directories as actually being directories. A '/' at the end of the line accounts for it.

c.f. |ignoring remove instruction for directory: greprefs/| which is echoed near the end of the make of update-packaging.

ewong has graciously accepted this bug over IRC, since I am too overtired to have any confidence in it.

when this lands, please land on both comm-central default branch and comm-central |COMM20b11_20110203_RELBRANCH| as well, so I can hopefully get our build2 out for testing within 24 hours of this bug being filed.
Status: NEW → ASSIGNED
Attachment #510928 - Flags: feedback?(bugspam.Callek)
Comment on attachment 510928 [details] [diff] [review]
Fixed paths in removed-files.in.

my skim looks good, I won't trust a real review from myself until I've rested, maybe KaiRo can get to this faster though
Attachment #510928 - Flags: review?(kairo)
Attachment #510928 - Flags: feedback?(bugspam.Callek)
Attachment #510928 - Flags: feedback+
Comment on attachment 510928 [details] [diff] [review]
Fixed paths in removed-files.in.

I filtered removed-files.in content out and that's what remained ;-)
r+, if I may.
Attachment #510928 - Flags: review?(kairo) → review+
Flags: in-testsuite-
Target Milestone: --- → seamonkey2.1b2
Comment on attachment 510928 [details] [diff] [review]
Fixed paths in removed-files.in.

I trust you haven't missed any, and those that are there look good.

Serge, as much as I like you taking a look at it, I don't think you are a peer of the build system yet, so it's better if this has my (or Callek's) stamp on it.
Attachment #510928 - Flags: review+
(In reply to comment #4)
> Comment on attachment 510928 [details] [diff] [review]
> Fixed paths in removed-files.in.
> 
> I trust you haven't missed any, and those that are there look good.

I'm uncertain on the Mac ones though.  That DefaultPlugin here
looks suspicious: 
http://mxr.mozilla.org/comm-central/source/suite/installer/removed-files.in#217
(In reply to comment #4)
> Comment on attachment 510928 [details] [diff] [review]
> Fixed paths in removed-files.in.
> 
> I trust you haven't missed any, and those that are there look good.

And this line confuses me:

http://mxr.mozilla.org/comm-central/source/suite/installer/removed-files.in#130
(In reply to comment #5)
> I'm uncertain on the Mac ones though.  That DefaultPlugin here
> looks suspicious: 
> http://mxr.mozilla.org/comm-central/source/suite/installer/removed-files.in#217

I *think* that's a file.

(In reply to comment #6)
> And this line confuses me:
> 
> http://mxr.mozilla.org/comm-central/source/suite/installer/removed-files.in#130

Mac has "app bundles" inside directories - those (.app or even .plugin) *look* like files even in the "Finder" file browser on the machine, but in reality are directories. So this one is correct.
(In reply to comment #7)
> > http://mxr.mozilla.org/comm-central/source/suite/installer/removed-files.in#217
> 
> I *think* that's a file.

Fwiw, this one should be the same (either way) as
202 ../Plug-Ins/PrintPDE.plugin/Contents/MacOS/PrintPDE
and I too assumed they are (the) files. (Though I know nothing about OSX.)
Pushed to comm-central COMM20b11_20110203_RELBRANCH
http://hg.mozilla.org/comm-central/rev/402a29ffb09b

Pushed to comm-central default
http://hg.mozilla.org/comm-central/rev/5dfe22893e47

Leaving to drivers to close.
Status: ASSIGNED → RESOLVED
Closed: 14 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: