Skip to content

Conversation

@djs55
Copy link
Contributor

@djs55 djs55 commented Mar 26, 2014

This is the same as other xapi daemons. It allows each option to
be read from a config file and overriden on the command-line.

Also, extend the default xapi.conf with comments describing all
available options which previously were undocumented.

Signed-off-by: Jon Ludlam [email protected]
Signed-off-by: David Scott [email protected]

@thomassa
Copy link
Contributor

thomassa commented Apr 7, 2014

@xen-git retest this please

1 similar comment
@djs55
Copy link
Contributor Author

djs55 commented Apr 24, 2014

@xen-git retest this please

@djs55
Copy link
Contributor Author

djs55 commented Apr 24, 2014

omake -j 8 phase3
*** omake: reading OMakefiles
*** omake: finished reading OMakefiles (0.27 sec)
*** omake: 95/1534 targets are up to date
*** omake: failed (0.50 sec, 0/0 scans, 0/0 rules, 0/521 digests)
*** omake error:
   File /usr/lib64/omake/build/OCaml.om: line 1041, characters 4-70
   Do not know how to build "ocaml/util/config_shared.cmx" required for "ocaml/xapi/xapi.opt"
make[4]: *** [all] Error 123

@djs55
Copy link
Contributor Author

djs55 commented Apr 25, 2014

Processing files: xapi-core-0.2-6586.x86_64
error: File must begin with "/": @ETCDIR@/udhcpd.skel


RPM build errors:
    File must begin with "/": @ETCDIR@/udhcpd.skel
make[3]: *** [/output/api/cli-regress.tar.gz] Error 1

Jon Ludlam and others added 7 commits April 28, 2014 14:29
This is the same as other xapi daemons. It allows each option to
be read from a config file and overriden on the command-line.

Also, extend the default xapi.conf with comments describing all
available options which previously were undocumented.

Signed-off-by: Jon Ludlam <[email protected]>
Signed-off-by: David Scott <[email protected]>
We will eventually replace this spec file with one from the buildroot.
In the meantime we follow the existing convention.

Signed-off-by: David Scott <[email protected]>
djs55 added a commit that referenced this pull request Apr 28, 2014
Replace ad-hoc config file handling with Xcp_service.configure
@djs55 djs55 merged commit 0c91d33 into xapi-project:master Apr 28, 2014
@djs55 djs55 deleted the from-xenserver-core-xcp-services branch April 28, 2014 14:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants