Matthew Farrellee wrote:
Jeff Garzik wrote:
Matthew Farrellee wrote:
make check attempts to start cld, but cannot find it if it is not
already in PATH. This patch uses CLDC_PREFIX, possibly from
pkg-config, to detect the location of cld with configure.
Signed-off-by: Matthew Farrellee <[email protected]>
---
configure.ac | 19 +++++++++++++++++++
test/Makefile.am | 1 +
test/start-daemon | 20 +-------------------
test/start-daemon.real.in | 20 ++++++++++++++++++++
4 files changed, 41 insertions(+), 19 deletions(-)
create mode 100644 test/start-daemon.real.in
oh, FYI: this missed updating test/.gitignore to tell git to ignore the
generated file.
Typically you want to run "git status" after builds and such, to make
sure that git is aware of the status of all files (in repo, or ignored).
Running "git status" is a good habit to get into. At least for me, I
have sent out patches that accidentally omit newly added files, because
I forgot to run "git status".
Thanks!
Jeff
So, I do run git status, but my normal workflow does not include heavy use of
.gitignore.
I'm actually not a fan of this use of .gitignore, mostly because I like git
clean as a means to reset things.
I hear ya :) But we do have non-git tarball users, which implies that
"make distclean" should clean everything... which implies truly
needing to use 'git clean' is more a bug than a feature.
I wouldn't ask you to change your workflow, but 'git status' and 'make
distclean' are the norm for these projects (as well as the kernel, where
we have the actual git author critique us :))
Jeff
--
To unsubscribe from this list: send the line "unsubscribe hail-devel" in
the body of a message to [email protected]
More majordomo info at http://vger.kernel.org/majordomo-info.html
