[RFC] Add support for the Solaris platform
Carl Worth
cworth at cworth.org
Thu Jun 3 17:22:47 PDT 2010
On Fri, 30 Apr 2010 21:00:24 +0200, Tomas Carnecky <tom at dbservice.com> wrote:
> Like on Mac OS X, the linker doesn't automatically resolve
> dependencies.
Thanks for this fix, Tomas!
I like the fix described above, but you seem to have sent a patch with
three independent changes, only one of which was described. See below
for what I've done...
> notmuch: $(notmuch_client_modules) lib/libnotmuch.a
> - $(call quiet,CXX $(CFLAGS)) $^ $(FINAL_LIBNOTMUCH_LDFLAGS) -o $@
> + $(call quiet,CXX $(CFLAGS)) $^ $(FINAL_NOTMUCH_LDFLAGS) -o $@
That seems to be correct. Could you describe what problem you ran into
that motivated you to find this fix? I have not committed this piece.
> -printf "Checking for Mac OS X (for shared library)... "
> +printf "Checking which platform we are on... "
> if [ `uname` = "Darwin" ] ; then
> - printf "Yes.\n"
> - mac_os_x=1
> + printf "Mac OS X.\n"
> + platform=MACOSX
...
> -ifeq ($(MAC_OS_X),1)
> +ifeq ($(PLATFORM),MACOSX)
> LIBRARY_SUFFIX = dylib
> # On OS X, library version numbers go before suffix.
> LINKER_NAME = libnotmuch.$(LIBRARY_SUFFIX)
That hunk was all described by your original commit message. So I've
applied this part with your original commit message and will be pushing
it soon.
> +/* Helper functions to test if a given dirent is of a certain type
> + */
> +static int
> +_is_reg(const char *path, struct dirent *entry)
...
> @@ -372,7 +426,7 @@ add_files_recursive (notmuch_database_t *notmuch,
>
> if (! S_ISREG (st.st_mode))
> continue;
> - } else if (entry->d_type != DT_REG) {
> + } else if (!_is_reg(path, entry)) {
> continue;
> }
This all might be well and good, but is also not described by the commit
message. So I have not committed this either. If you resend with an
actual description of the change, I'll be happy to take a closer look.
Thanks again,
-Carl
--
carl.d.worth at intel.com
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: not available
URL: <http://notmuchmail.org/pipermail/notmuch/attachments/20100603/df6db626/attachment.pgp>
More information about the notmuch
mailing list