Closed Bug 398003 Opened 17 years ago Closed 17 years ago

Overhaul SeaMonkey README files

Categories

(SeaMonkey :: Installer, defect)

x86
Linux
defect
Not set
trivial

Tracking

(Not tracked)

RESOLVED FIXED
seamonkey2.0a1

People

(Reporter: kairo, Assigned: kairo)

Details

Attachments

(1 file)

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.
Actually, while working on this, I realized that we need to overhaul all of the README files we have, changing summary accordingly.
Summary: Make Linux README wrap at 80 characters → Overhaul SeaMonkey README files
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.
Attachment #282835 - Flags: review?(ajschult)
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)
Attachment #282835 - Flags: review?(ajschult) → review+
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).
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?
> 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.
checked in, fixed.
Status: NEW → RESOLVED
Closed: 17 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.