163354 - readme.txt and license.txt missing

Status: VERIFIED FIXED

(SeaMonkey :: Installer, defect)

Description

[alanjstr]

Mozilla/5.0 (Windows; U; Windows NT 5.0; en-US; rv:1.1b) Gecko/20020818

I did a "browser only" installation.  Shorcuts were created pointing to
readme.txt and license.txt in "C:\Program Files\mozilla.org\Mozilla".  Neither
file is in there.  There is no readme.txt at all under that directory tree, but
a license.txt exists in "C:\Program Files\mozilla.org\Mozilla\Setup\".

Comment 1

[Wesha]

Confirmed 2002081808/winXP

Comment 2

[Chris Collins]

Missing readme and license happens in NT 4.0 also.
There is a readme and license icon in
START->Program Files->Mozilla->readme.txt(and license)
right where is should be, but no files.

Comment 3

[Serge Gautherie (:sgautherie)]

With "Mozilla/5.0 (Windows; U; Win95; en-US; rv:1.2a) Gecko/20020910"

Same bug (after custom install).

Comment 4

[Matthias Versen [:Matti]]

-> Insaller (?)

Comment 5

[Matthias Versen [:Matti]]

*** Bug 169218 has been marked as a duplicate of this bug. ***

Comment 6

[Henrik Gemal]

*** Bug 175141 has been marked as a duplicate of this bug. ***

Comment 7

[Serge Gautherie (:sgautherie)]

"Mozilla/5.0 (Windows; U; Win95; en-US; rv:1.2b) Gecko/20021016"

Same bug (after custom install).

Comment 8

[Mike Kaply [:mkaply]]

Attachment: Fix for problem

Actually, this should be all that is needed.

We need to put the readme and license in the bin directory.

All the other code is already in place to make this happen.

Comment 9

[Mike Kaply [:mkaply]]

Adding seawood for review since this in an xpfe/bootstrap Makefile change.

Comment 10

[hacker formerly known as [e-mail]]

Comment on attachment 103747 [details] [diff] [review]
Fix for problem

If you don't need that OS2 ifdef, remove it before you check it in. r=cls

Comment 11

[Mike Kaply [:mkaply]]

I was told that README.TXT was only for the top three platforms, so if I wanted 
OS/2 readme stuff, it couldn't go there.

So other platforms that want READMEs will have to put them somewhere else and 
then have an if to copy them instead of README.TXT.

I put this there as kind of an example as to what the right thing to do would 
be.

I'm going to try to get permission to check in a readme.os2 so this code will 
actually work.

Comment 12

[David Baron :dbaron: (⌚️UTC-5, no longer working on Mozilla)]

Comment on attachment 103747 [details] [diff] [review]
Fix for problem

a=dbaron for trunk checkin

Comment 13

[Mike Kaply [:mkaply]]

This bug is fixed.

Verified on current nightly.

Comment 14

[Grace Bush]

verified on 1106 nightly

Comment 15

[Serge Gautherie (:sgautherie)]

"Mozilla/5.0 (Windows; U; Win95; en-US; rv:1.2) Gecko/20021126"

Fixed :-)