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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).