398003 - Overhaul SeaMonkey README files

Status: RESOLVED FIXED

(SeaMonkey :: Installer, defect)

Description

[Robert Kaiser]

As we're not using the Linux installer any more, which needed unwrapped lines for displaying the README inline, we should wrap the README we install into all our packages at 80 characters on Linux as well, just like we do for other platforms.

Comment 1

[Robert Kaiser]

Actually, while working on this, I realized that we need to overhaul all of the README files we have, changing summary accordingly.

Comment 2

[Robert Kaiser]

Attachment: Fix various README errors and glitches

This should fix wrapping on Linux, make us refer the new project page (which should be live at the new location before 2.0 final and now just redirects to the old one), link the correct build instructions, remove "Navigator" for bug 397846 and correct system requirements.

Comment 3

[Andrew Schultz]

Comment on attachment 282835 [details] [diff] [review]
Fix various README errors and glitches

>+To submit bugs or other feedback, see the browser's QA menu and check out

As discussed on IRC, we should probably not reference the QA menu since it won't be there in release builds (and the file doesn't get maintained enough to update it after we switch the QA menu off).  A link to the "Getting Involved" page should suffice.

>+     Red Hat Enterprise Linux 5.0, Fedora Core 6, ubuntu 6.10 ("Edgy"),

Red Hat Enterprise Linux 5

They don't seem to use decimal versions anymore (we run "Red Hat Enterprise Linux WS release 4 (Nahant Update 5)" at work)

Comment 4

[Andrew Schultz]

Comment on attachment 282835 [details] [diff] [review]
Fix various README errors and glitches

>+   - The following library versions (or compatible) are required:
>+     glibc 2.3.2, X.org 1.x, GTK 2.x, fontconfig, pango 1.10, libstdc++ 6.

You might also note that curl is needed for crashreporter.

>+     Red Hat Enterprise Linux 5.0, Fedora Core 6, ubuntu 6.10 ("Edgy"),
>+     Debian "Lenny" (testing), and openSUSE 10.2 (or later) installations
>+     should work.

I suspect most of these will have the right version of curl.  Later distros might not (Fedora 7's is too new).

Comment 5

[Robert Kaiser]

OK, changed the QA menu stuff to point to http://www.seamonkey-project.org/dev/get-involved instead, and removed the .0 from RHEL.

I'm a bit unsure how to add this requirement of curl for crashreporter, I think that most Linux systems should have some version of it installed anyways. That issue with newer versions of curl is a known bug and AFAIK, lusr and others have been working on getting it resolved, not sure if they did succeed yet.

Someone mentioned on IRC that the 256 color stuff about Modern theme also should probably go, are you OK with removing the whole "* General" chapter from system requirements?
Should I attach a new patch with those changes and re-request review, or are you OK with just going with the changes I mentioned here?

Comment 6

[Andrew Schultz]

> Someone mentioned on IRC that the 256 color stuff about Modern theme also
> should probably go, are you OK with removing the whole "* General" chapter from
> system requirements?

That was me.  :)  So, yes, r=me with the changes you mentioned.

Comment 7

[Robert Kaiser]

checked in, fixed.