From: Robert Peterson <rpeterso@redhat.com>
To: cluster-devel.redhat.com
Subject: [Cluster-devel] [PATCH] cleanup build system a bit.
Date: Fri, 27 Apr 2007 07:52:56 -0500 [thread overview]
Message-ID: <4631F228.3090906@redhat.com> (raw)
In-Reply-To: <463019C3.8090302@ubuntu.com>
Fabio Massimo Di Nitto wrote:
>> I think this patch is something that we can commit to the -HEAD branch and start
>> using for FC8 and beyond. We shouldn't have a problem patching up any minor
>> issues that crop up. I vote for committing this patch.
Hi Fabio,
This looks like a good patch to me. I've been too busy to look at it in
any great detail, but I did look at the highlights. I have a minor concern
about the Makefile in gfs2/mkfs: IIRC, it looked like the default "Make"
(all) specified only mkfs.gfs2 as the target, whereas we need to be sure to
that gfs2_jadd and gfs2_grow get built as well. Er, well, I guess you don't
have gfs2_grow yet (but I do). Maybe it's already assured with some magic
and trickery, as it often happens in many Makefiles, but that will be apparent
soon enough if/when I get a chance to try it.
As with any change with so many things affected, there might be some minor
glitches or gotchas that we'll find, but I think we can work around those
as we encounter them. Good job!
Regards,
Bob Peterson
Red Hat Cluster Suite
next prev parent reply other threads:[~2007-04-27 12:52 UTC|newest]
Thread overview: 18+ messages / expand[flat|nested] mbox.gz Atom feed top
2007-04-24 12:56 [Cluster-devel] [PATCH] cleanup build system a bit Fabio Massimo Di Nitto
2007-04-24 13:34 ` Fabio Massimo Di Nitto
2007-04-25 18:05 ` Lon Hohberger
2007-04-25 18:29 ` Fabio Massimo Di Nitto
2007-04-25 21:13 ` Chris Feist
2007-04-26 3:17 ` Fabio Massimo Di Nitto
2007-04-27 12:52 ` Robert Peterson [this message]
2007-04-27 13:15 ` Fabio Massimo Di Nitto
2007-04-26 16:55 ` Fabio Massimo Di Nitto
2007-04-27 9:24 ` Fabio Massimo Di Nitto
2007-04-27 9:28 ` [Cluster-devel] [PATCH 001/008] Fix configure help typo Fabio Massimo Di Nitto
2007-04-27 9:29 ` [Cluster-devel] [PATCH 002/008] Fabio Massimo Di Nitto
2007-04-27 9:30 ` [Cluster-devel] [PATCH 003/008] Fix --release_* check Fabio Massimo Di Nitto
2007-04-27 9:32 ` [Cluster-devel] [PATCH 004/008] gnbd_export: convert harcoded DEFAULT_GETUID to build flag Fabio Massimo Di Nitto
2007-04-27 9:33 ` [Cluster-devel] [PATCH 005/008] Fix userland: target depends Fabio Massimo Di Nitto
2007-04-27 9:34 ` [Cluster-devel] [PATCH 006/008] Fix make all target in passtrough Makefiles and make them more consistent Fabio Massimo Di Nitto
2007-04-27 9:35 ` [Cluster-devel] [PATCH 007/008] Readd -Werror to rgmanager dropped by mistake in the big cleanup Fabio Massimo Di Nitto
2007-04-27 9:36 ` [Cluster-devel] [PATCH 008/008] Readd SHAREDIR define to rgmanager/src/daemons/Makefile Fabio Massimo Di Nitto
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=4631F228.3090906@redhat.com \
--to=rpeterso@redhat.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.