Docs missing prerequsite for required MIME::Base64 and MIME::Parser

RESOLVED FIXED in Bugzilla 2.22

Status

()

P3
minor
RESOLVED FIXED
14 years ago
13 years ago

People

(Reporter: bugreport, Assigned: spam)

Tracking

2.21
Bugzilla 2.22

Details

Attachments

(1 attachment, 4 obsolete attachments)

(Reporter)

Description

14 years ago
Docs need update
(Reporter)

Updated

14 years ago
Severity: normal → minor
Priority: -- → P3
Target Milestone: --- → Bugzilla 2.22
Created attachment 206696 [details] [diff] [review]
patch for tip

coordinate to checksetup.pl
Attachment #206696 - Flags: review?
Attachment #206696 - Flags: review? → review?(documentation)

Updated

13 years ago
Assignee: documentation → bmo
Comment on attachment 206696 [details] [diff] [review]
patch for tip

>Index: docs/xml/Bugzilla-Guide.xml

This looks fine...

>Index: docs/xml/installation.xml
     
>+            <link linkend="install-modules-mime-parser">MIME::Base64</link>

>+            <link linkend="install-modules-mime-parser">MIME::Parser</link>

>+            <link linkend="install-modules-image-magick">Image::Magick</link>
>+            (&min-image-magick-ver;) for convert BMP image attachments to PNG

Don't need the <link> elements... there's nothing to link to.

"for convert BMP" -> "for converting BMP"
Attachment #206696 - Flags: review?(documentation) → review-
Created attachment 206865 [details] [diff] [review]
v2

apply comments
Attachment #206696 - Attachment is obsolete: true
Attachment #206865 - Flags: review?(documentation)
Comment on attachment 206865 [details] [diff] [review]
v2

>Index: docs/xml/installation.xml

>+
>+        <listitem>
>+          <para>
>+            MIME::Base64 (&min-mime-base64-ver;) for utf-8 encoding
>+          </para>
>+        </listitem>
>+
>+        <listitem>
>+          <para>
>+            MIME::Parser (&min-mime-parser-ver;) for email interface
>+          </para>
>+        </listitem>
>+

Can you kill the descriptions, since none of the other required modules have them?

Also, you forgot the Bugzilla-Guide.xml change in this patch ;)
Attachment #206865 - Flags: review?(documentation) → review-
Created attachment 206886 [details] [diff] [review]
patch for tip v3

argh..
perhaps I should have been slept at that time
Attachment #206865 - Attachment is obsolete: true
Attachment #206886 - Flags: review?(documentation)
Created attachment 206904 [details] [diff] [review]
added modules.xml
Attachment #206886 - Attachment is obsolete: true
Attachment #206904 - Flags: review?(documentation)
Attachment #206886 - Flags: review?(documentation)
Comment on attachment 206904 [details] [diff] [review]
added modules.xml

>Index: docs/xml/Bugzilla-Guide.xml

This is still fine :)

>Index: docs/xml/installation.xml

>         <listitem>
>           <para>
>-            <link linkend="install-modules-mime-parser">MIME::Parser</link>
>-            (&min-mime-parser-ver;) for the optional email interface
>+            Image::Magick (&min-image-magick-ver;) for converting BMP image attachments to PNG
>           </para>
>         </listitem>

I wonder if this needs a <section> like that for PatchReader just explaining that the mdoule is optional, and only required if you want to convert the images...

Thoughts?

>Index: docs/xml/modules.xml

>+   <para>
>+      Image::Magick:
>+      <literallayout>
>+        CPAN Download Page: <ulink url="http://search.cpan.org/dist/XML-Parser/"/>
>+        PPM Download Link: Included in Windows binary package.
>+        Documentation: <ulink url="http://www.imagemagick.org/script/resources.php"/>
>+      </literallayout>
>+    </para>

As per IRC, this should be http://search.cpan.org/dist/PerlMagick/ instead.
Attachment #206904 - Flags: review?(documentation) → review-
Created attachment 207000 [details] [diff] [review]
sorrrry for wrong patch and bug spam

correct information should be put there now, and bringing together those modules can be done when someone decide how to do.
so i don't touch about that right now ;-)
Attachment #206904 - Attachment is obsolete: true
Attachment #207000 - Flags: review?(documentation)
Comment on attachment 207000 [details] [diff] [review]
sorrrry for wrong patch and bug spam

Looks good to me.
Attachment #207000 - Flags: review?(documentation) → review+
Checked in:

Checking in docs/xml/Bugzilla-Guide.xml;
/cvsroot/mozilla/webtools/bugzilla/docs/xml/Bugzilla-Guide.xml,v  <--  Bugzilla-Guide.xml
new revision: 1.56; previous revision: 1.55
done
Checking in docs/xml/installation.xml;
/cvsroot/mozilla/webtools/bugzilla/docs/xml/installation.xml,v  <--  installation.xml
new revision: 1.107; previous revision: 1.106
done
Checking in docs/xml/modules.xml;
/cvsroot/mozilla/webtools/bugzilla/docs/xml/modules.xml,v  <--  modules.xml
new revision: 1.4; previous revision: 1.3
done
Status: NEW → RESOLVED
Last Resolved: 13 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.