[ptxdist] [PATCH] Do not copy the kernel config file if it is not really changed

2010-06-07 Thread Remy Bohmer
The kernel config file only needs to be copied if: * lines not starting with '#' are changed. * lines starting with '# C' are changed to detect disabled options. Signed-off-by: Remy Bohmer --- rules/kernel.make |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/rules/kerne

Re: [ptxdist] [PATCH v2 02/20] Get rid of toolchain check-error message

2010-06-07 Thread Tim Sander
Hi > Yes, that is possible, as long as the required toolchain is in the PATH. > > > If not, I think there should be an aditional warning if it's missing. > > Otherwise this patch is ok. > > Marc, what do you think? I don't like the idea that the toolchain link can be removed completely. By the id

[ptxdist] [PATCH 1/1] lighttpd: add upstart support

2010-06-07 Thread Tim Sander
Hi Please consider this patch for master. As always mail due to technical dificulties for review only, please use git repo at: git://gitorious.org/~tstone/oselas/ptxdist-upstart.git for merging. Best regards Tim Signed-Off-By: Tim Sander diff --git a/generic/etc/init/lighttpd.conf b/generic/et

Re: [ptxdist] [PATCH v2] Fix dependencies to files in projectroot

2010-06-07 Thread Michael Olbrich
On Sat, Jun 05, 2010 at 12:08:34PM +0200, Remy Bohmer wrote: > When files in the projectroot directory are being used in install_copy() and > install_alternative() then there are no dependencies to these files. > > This is being solved by storing all dependencies of the targetinstall > stage in a

Re: [ptxdist] [PATCH v2] Conditional include do not require if's

2010-06-07 Thread Michael Olbrich
On Sat, Jun 05, 2010 at 10:33:47AM +0200, Remy Bohmer wrote: > To check for the existence of certain files and include them if these > exist can be done by just simply prepending the include directive with > a '-' sign, without if and wildcard checks. I looked at the Make dokumentation to see wha

Re: [ptxdist] [PATCH v2 01/20] Do not fail if provided kernel-config file is read-only

2010-06-07 Thread Michael Olbrich
On Mon, Jun 07, 2010 at 11:13:25AM +0200, Remy Bohmer wrote: > 2010/5/31 Marc Kleine-Budde : > > Remy Bohmer wrote: > >>> I don't like this. $(KERNEL_CONFIG) may not be readonly on purpose and > >>> changes get lost. What about: > >>> > >>> if diff -q  "$(KERNEL_DIR)/.config" "$(<)" > /dev/null; th

Re: [ptxdist] [PATCH v2 01/20] Do not fail if provided kernel-config file is read-only

2010-06-07 Thread Remy Bohmer
Hi, 2010/5/31 Marc Kleine-Budde : > Remy Bohmer wrote: >> Hi, >>> I don't like this. $(KERNEL_CONFIG) may not be readonly on purpose and >>> changes get lost. What about: >>> >>> if diff -q  "$(KERNEL_DIR)/.config" "$(<)" > /dev/null; then >>>        cp "$(KERNEL_DIR)/.config" "$(<)" >>> fi >>> >>