apple / ccs-calendarserver

The Calendar and Contacts Server.
https://www.calendarserver.org
Apache License 2.0
486 stars 136 forks source link

run script should parse_options() before including build.sh #313

Closed macosforgebot closed 14 years ago

macosforgebot commented 14 years ago

software@… originally submitted this as ticket:399


I am trying to install a carddav server, and after building the sources, I get this error:

./run -a

Missing config file: /home/freek/calendarserver/conf/caldavd-dev.plist
You might want to start by copying the test configuration:

  cp conf/carddavd-test.plist conf/carddavd-dev.plist

This is obviously wrong (caldav-dev.plist is missing, but I need to cp cardav-dev.plist?)

The reasons is that the first line is defined in support/build.sh, which defines $conf as "${wd}/conf/${DAVD}davd-dev.plist";. However, build.sh is included (and executed!) before run has set $DAVD correctly.

The solution is to to first let run set $DAVD, before including build.sh

Patch is attached.

macosforgebot commented 14 years ago

software@… originally submitted this as attachment:run-patch.diff:⁠ticket:399


Patch

macosforgebot commented 14 years ago

@glyph originally submitted this as comment:1:⁠ticket:399


That won't work, as the "init_build" at the end of build.sh will just wipe out the options parsed by parse_options.

While the ordering could be better here, the right thing to do is just to get rid of the '-a' option, as it's no longer necessary; you can run a caldav/carddav combo server, or select carddav entirely via the plist. No command-line options are necessary to distinguish between the two, the '-a' option was simply a stopgap as we were working on merging them.

macosforgebot commented 14 years ago

@wsanchez originally submitted this as comment:2:⁠ticket:399


-a is gone in r6385