It doesn't need to be writable to fix the tests, Jan tracked the real problem down. What's happening is we hit ensure_cookie_auth_secret/1 during the cookie_auth.js test, which fails to write to the read-only .ini file. Instead, we should explicitly set up the cookie auth secret and pass the non-persistent flag to couch_config. Then the test passes and all the files can stay read-only.
As to the revert^3, one can only imagine what goes through Jan's mind sometimes... :) B. On 30 April 2012 17:26, Paul Davis <[email protected]> wrote: > This series of commits confuses me greatly. There are a lot of reverts > and the explanation of it breaking compilation from read only media is > incorrect. > > This file should be writable or ./utils/run in a VPATH build can't > edit its configuration which is incorrect. Also it breaks the tests. > > On Mon, Apr 30, 2012 at 6:24 AM, Jan Lehnardt <[email protected]> wrote: >> Sorry :) >> >> On Apr 30, 2012, at 12:23 , [email protected] wrote: >> >>> Updated Branches: >>> refs/heads/master bdcfcca1e -> 9acc34ed9 >>> >>> >>> Revert "Revert "Revert "Make sure that local_dev.ini is writable""" >>> >>> Revert. Git <3. >>> >>> This reverts commit bdcfcca1ec92f379e5376741e5ab103630573ac4. >>> >>> >>> Project: http://git-wip-us.apache.org/repos/asf/couchdb/repo >>> Commit: http://git-wip-us.apache.org/repos/asf/couchdb/commit/9acc34ed >>> Tree: http://git-wip-us.apache.org/repos/asf/couchdb/tree/9acc34ed >>> Diff: http://git-wip-us.apache.org/repos/asf/couchdb/diff/9acc34ed >>> >>> Branch: refs/heads/master >>> Commit: 9acc34ed9ed4315f7cd6f5b8ac45e47f2c0b8cb6 >>> Parents: bdcfcca1 >>> Author: Jan Lehnardt <[email protected]> >>> Authored: Mon Apr 30 12:23:00 2012 +0100 >>> Committer: Jan Lehnardt <[email protected]> >>> Committed: Mon Apr 30 12:23:00 2012 +0100 >>> >>> ---------------------------------------------------------------------- >>> etc/couchdb/Makefile.am | 1 - >>> 1 files changed, 0 insertions(+), 1 deletions(-) >>> ---------------------------------------------------------------------- >>> >>> >>> http://git-wip-us.apache.org/repos/asf/couchdb/blob/9acc34ed/etc/couchdb/Makefile.am >>> ---------------------------------------------------------------------- >>> diff --git a/etc/couchdb/Makefile.am b/etc/couchdb/Makefile.am >>> index fe045f9..dd1054b 100644 >>> --- a/etc/couchdb/Makefile.am >>> +++ b/etc/couchdb/Makefile.am >>> @@ -73,7 +73,6 @@ default_dev.ini: default.ini.tpl >>> local_dev.ini: local.ini >>> if test ! -f "$@"; then \ >>> cp $< $@; \ >>> - chmod +w $@; \ >>> fi >>> >>> install-data-hook: >>> >>
