Ok. I will update the patch as per suggestions below and then try to check in. 
If there are any other corrections or suggestions with regards to the patch or 
the content of the updates - please let me know.

> -----Original Message-----
> From: Richard Quadling [mailto:rquadl...@googlemail.com]
> Sent: Wednesday, October 14, 2009 12:33 PM
> To: Ruslan Yakushev
> Cc: PHP Documentation ML
> Subject: Re: [PHP-DOC] Patch for the Windows Installation docs on
> php.net
> 
> 2009/10/14 Ruslan Yakushev <rusl...@microsoft.com>:
> > Thanks Richard,
> >
> > Do you want me to produce an updated patch, or you can make these
> minor changes yourself before submitting?
> >
> >
> >
> >> -----Original Message-----
> >> From: Richard Quadling [mailto:rquadl...@googlemail.com]
> >> Sent: Wednesday, October 14, 2009 2:23 AM
> >> To: Ruslan Yakushev
> >> Cc: PHP Documentation ML
> >> Subject: Re: [PHP-DOC] Patch for the Windows Installation docs on
> >> php.net
> >>
> >> 2009/10/13 Ruslan Yakushev <rusl...@microsoft.com>:
> >> > Hello phpdoc team,
> >> >
> >> > The first patch for the windows specific docs on php.net is ready.
> >> The patch is available here:
> http://ruslany.net/download/phpdocs.zip.
> >> It contains the following changes:
> >> >
> >> > 1. New article "Microsoft IIS 5.1 and IIS 6.0"
> >> > 2. New article "Microsoft IIS 7.0 and later"
> >> > 3. Updated article "Manual Installation Steps"
> >> > 4. Removed article "Microsoft IIS"
> >> >
> >> > Also the package contains the "screenshots" folder that needs to
> be
> >> placed into the en\install\windows directory before the docs are
> built.
> >> >
> >> > I would appreciate any feedback and suggestions related to the
> >> content of the patch. If the patch looks good - what is the process
> for
> >> submitting the changes?
> >> >
> >> > Thanks,
> >> > Ruslan
> >> >
> >>
> >> Hi.
> >>
> >> A couple of minor things ...
> >>
> >> 1 - BOM is present. Shouldn't be an issue, but is normally not used.
> >>
> >> 2 - For the VIM block at the bottom of the file ...
> >>
> >> sgml-default-dtd-file:"../../manual.ced"
> >>
> >> is now ...
> >>
> >> sgml-default-dtd-file:"~/.phpdoc/manual.ced"
> >>
> >>
> >> 3 - The screenshots folder is called figures for all the other
> >> extensions. As the images are copied to a single location, it may
> not
> >> make any difference, but for consistency ... (I like consistency).
> >>
> >>
> >> 4 - In en/install/windows/index.xml, the line &install.windows.iis;
> >> should be removed.
> >>
> >>
> >>
> >> But, I added the patch to my checkout, edited the index.xml file and
> >> successfully built the CHM file.
> >>
> >>
> >>
> >>
> >> As for submitting it, I can do that, but I'll wait to see if Hannes,
> >> et al has anything additional to say.
> >>
> >>
> >> Regards,
> >>
> >> Richard.
> >> --
> >> -----
> >> Richard Quadling
> >> "Standing on the shoulders of some very clever giants!"
> >> EE : http://www.experts-exchange.com/M_248814.html
> >> Zend Certified Engineer :
> >> http://zend.com/zce.php?c=ZEND002498&r=213474731
> >> ZOPA : http://uk.zopa.com/member/RQuadling
> >
> >
> 
> You've got the power!
> 
> --
> -----
> Richard Quadling
> "Standing on the shoulders of some very clever giants!"
> EE : http://www.experts-exchange.com/M_248814.html
> Zend Certified Engineer :
> http://zend.com/zce.php?c=ZEND002498&r=213474731
> ZOPA : http://uk.zopa.com/member/RQuadling

Reply via email to