From: "Joachim Schmitz" <jojo@schmitz-digital•de>
To: "'Junio C Hamano'" <gitster@pobox•com>
Cc: "'Shawn Pearce'" <spearce@spearce•org>, <git@vger•kernel.org>,
<rsbecker@nexbridge•com>
Subject: RE: Porting git to HP NonStop
Date: Mon, 20 Aug 2012 12:22:03 +0200 [thread overview]
Message-ID: <000601cd7ebd$a4ef5740$eece05c0$@schmitz-digital.de> (raw)
In-Reply-To: <7v628epzia.fsf@alter.siamese.dyndns.org>
> From: Junio C Hamano [mailto:gitster@pobox•com]
> Sent: Sunday, August 19, 2012 7:23 PM
> To: Joachim Schmitz
> Cc: 'Shawn Pearce'; git@vger•kernel.org; rsbecker@nexbridge•com
> Subject: Re: Porting git to HP NonStop
>
> "Joachim Schmitz" <jojo@schmitz-digital•de> writes:
>
> > Found the problem: our mkdir(dir,flags) fails with ENOENT when dir
> > ends with a '/'.
> > Not sure whether this us a bug on out platform or just allowed by
> > POSIX and as such a wrong assumption in git though?
> >
> > [shortly after]
> > A bit of googleing revealed that there is a GNUlib solution for this,
> > which claims that at least NetBSD 1.5.2 has the same problem.
> > (http://www.opensource.apple.com/source/gpatch/gpatch-2/patch/mkdir.c)
> >
> > And apparently this has been discussed on the git mailing list too, 2
> > years
> > ago:
> > http://lists-archives.com/git/728359-git-s-use-of-mkdir-2.html,
> > there's a patch too.
>
> Given that newer BSDs have fixed libc to accept directory name with a
trailing
> slash, and that we use mkdir(2) in many places, I think the right way to
do so is
> still what I suggested in that old thread in the last paragraph of my
message
>
> http://thread.gmane.org/gmane.comp.version-
> control.git/155812/focus=155876
>
> That is, have compat/tandem.c and define a replacement mkdir(2) in a way
> similar to how MinGW does so.
OK, I'll go for a compat/mkdir.c though.
We shouldn't call it tandem.c as Tandem, the Company, doesn't exist anymore
and since more than a decade (bough by Compaq, then HP), only the __TANDEM
survived in our compiler and headers/libraries. Could call it NonStop.c, but
I don't really like that idea either, I'd rather keep it more generic, just
in case someone else might need it too, or that issue someday gets fixed for
NonStop.
> > For now I've fixed it like this:
> > /usr/local/bin/diff -EBbu ./builtin/init-db.c.orig ./builtin/init-db.c
> > --- ./builtin/init-db.c.orig 2012-08-19 03:55:50 -0500
> > +++ ./builtin/init-db.c 2012-08-19 03:39:57 -0500
> > @@ -25,7 +25,16 @@
> >
> > static void safe_create_dir(const char *dir, int share) {
> > +#ifdef __TANDEM /* our mkdir() can't cope with a trailing '/' */
> > + char mydir[PATH_MAX];
> > +
> > + strcpy(mydir,dir);
> > + if (dir[strlen(dir)-1] == '/')
> > + mydir[strlen(dir)-1] = '\0';
> > + if (mkdir(mydir, 0777) < 0) {
> > +#else
> > if (mkdir(dir, 0777) < 0) {
> > +#endif
>
> Move that part inside #ifdef __TANDEM to define
>
> int tandem_mkdir(const char *dir, mode_t mode)
> {
> ...
> }
I'll go for git_mkdir(), similar to other git wrappers, (like for mmap,
pread, fopen, snprintf, vsnprintf, qsort). Could call it gitmkdir() too
(like for basename, setenv, mkdtemp, mkstemps, unsetenv, strcasestr,
strlcpy, strtoumax, strtoimax, strtok_r, hstrerror, memmem, strchrnul,
memcpy), Opinions?
It seems the ones without the "_" are for missing APIs and the ones with "_"
to wrap existing APIs (not sure about mmap and pread)?
Here it's current state:
$ cat compat/mkdir.c
#include "../git-compat-util.h"
#undef mkdir
/* for platforms that can't deal with a trailing '/' */
int git_mkdir(const char *dir, mode_t mode)
{
int retval;
char *tmp_dir = NULL;
size_t len = strlen(dir);
if (len && dir[len-1] == '/') {
if ((tmp_dir = strdup(dir)) == NULL)
return -1;
tmp_dir[len-1] = '\0';
}
else
tmp_dir = (char *)dir;
retval = mkdir(tmp_dir, mode);
if (tmp_dir != dir)
free(tmp_dir);
return retval;
}
$
There is room for improvement though: it only removes one trailing slash. By
far not as advanced and generic as GNUlib's mkdir wrapper, but should be
good enough for git's usage.
> in your new file compat/tandem.c, add
>
> #ifdef __TANDEM
> #define mkdir(a,b) tandem_mkdir((a), (b))
> #endif
>
> to git-compat-util.h
Again, git_mkdir, see above
> and then add compat/tandem.o to COMPAT_OBJS in the
> top-level Makefile.
For now I've added it to the (new) NOSTOP_KERNEL section.
We may want it to go along with some
MKDIR_DISLIKES_TRAILING_SLASH or MKDIR_BOGUS_TRAILING_SLASH some such.
Opinions, Ideas?
> That way we do not have to keep an ugly platform specific ifdef in the
very
> generic codepath.
Agreed, it was my quick and dirty fix for it.
Bye, Jojo
next prev parent reply other threads:[~2012-08-20 10:22 UTC|newest]
Thread overview: 39+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-08-10 15:04 Porting git to HP NonStop Joachim Schmitz
2012-08-10 16:27 ` Shawn Pearce
2012-08-10 17:32 ` Joachim Schmitz
2012-08-10 17:38 ` Shawn Pearce
2012-08-19 8:57 ` Joachim Schmitz
2012-08-19 17:23 ` Junio C Hamano
2012-08-20 10:22 ` Joachim Schmitz [this message]
2012-08-20 14:41 ` Junio C Hamano
2012-08-20 16:09 ` Joachim Schmitz
2012-08-20 16:53 ` Junio C Hamano
2012-08-22 16:30 ` Joachim Schmitz
2012-08-22 17:00 ` Brandon Casey
2012-08-22 17:13 ` Brandon Casey
2012-08-22 17:18 ` Joachim Schmitz
2012-08-22 17:23 ` Brandon Casey
2012-08-22 17:30 ` Joachim Schmitz
2012-08-22 17:30 ` Junio C Hamano
2012-08-22 18:01 ` Joachim Schmitz
2012-08-22 18:24 ` Junio C Hamano
2012-08-22 18:52 ` Joachim Schmitz
2012-08-22 17:41 ` Johannes Sixt
2012-08-22 18:02 ` Joachim Schmitz
2012-08-22 18:09 ` Johannes Sixt
2012-08-22 18:18 ` Joachim Schmitz
2012-08-22 18:09 ` Brandon Casey
2012-08-22 18:24 ` Brandon Casey
2012-08-22 18:33 ` Junio C Hamano
2012-08-22 18:38 ` Brandon Casey
2012-08-22 20:18 ` Joachim Schmitz
2012-08-22 20:49 ` Junio C Hamano
2012-08-22 21:05 ` Joachim Schmitz
2012-08-22 21:12 ` Junio C Hamano
2012-08-22 21:22 ` Joachim Schmitz
2012-08-22 21:54 ` Junio C Hamano
2012-08-22 18:26 ` Junio C Hamano
2012-08-10 20:08 ` Joachim Schmitz
2012-08-11 8:20 ` Johannes Sixt
2012-08-14 7:05 ` Joachim Schmitz
2012-08-14 14:56 ` Junio C Hamano
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='000601cd7ebd$a4ef5740$eece05c0$@schmitz-digital.de' \
--to=jojo@schmitz-digital$(echo .)de \
--cc=git@vger$(echo .)kernel.org \
--cc=gitster@pobox$(echo .)com \
--cc=rsbecker@nexbridge$(echo .)com \
--cc=spearce@spearce$(echo .)org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
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