[ 
https://issues.apache.org/jira/browse/COUCHDB-889?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12912304#action_12912304
 ] 

Mark Hammond commented on COUCHDB-889:
--------------------------------------

Many of the changes look good, but have the following comments:

* I wonder if including today's current versions is a good thing - it seems it 
will just increase the bitrot in the future.  Can we just default to "latest 
available" except in the cases where we know it is not?  If not, then we 
probably can't justify the existing "latest available" references either.

* Where is nsis used?

* The inclusion of your entire PATH doesn't seem to add much value either - if 
the instructions are correct the path will be correct - so something is 
redundant.  Less is more when it comes to busy people trying to get a build up, 
and the specified "perfect path" will be incorrect if the retail version of 
MSVC is used, for example.

* Finally, using seamonkey instead of spidermonkey is a fair bit more effort 
wrt compilation - it might be reasonable to note that spidermonkey can be used 
if the reader can decypher the build instructions, or at least indicate 
something like "almost all mozilla products will build the spidermonkey we need 
(and spidermonkey can, with some difficulty, even be built stand-alone)  - 
below are instructions for seamonkey, but get a spidermonkey using whatever 
technique you like"

> improved docs for windows compile from source in INSTALL.Windows
> ----------------------------------------------------------------
>
>                 Key: COUCHDB-889
>                 URL: https://issues.apache.org/jira/browse/COUCHDB-889
>             Project: CouchDB
>          Issue Type: Improvement
>          Components: Build System, Documentation
>    Affects Versions: 0.11.2, 1.0.1
>         Environment: Windows only.
>            Reporter: Dave Cottlehuber
>             Fix For: 0.11.3, 1.0.2
>
>         Attachments: windows_build_from_source_docs.patch
>
>   Original Estimate: 0h
>  Remaining Estimate: 0h
>
> ./INSTALL.Windows does not have enough detail to compile from source, due to 
> internet bit rot. 
> Updates include -
> - clarification on versions for 32-bit and 64-bit compile setup
> - using free Microsoft Visual Studio 2008 Express C++ compiler instead of 
> full commercial release
> - improved details on building javascript, libcurl from source

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

Reply via email to