On 23/10/29 08:49PM, Markus Wichmann wrote:
> Am Sun, Oct 29, 2023 at 06:00:18PM +0100 schrieb Страхиња Радић:
> > mkdir returning EISDIR is definitely not a part of POSIX.[1]
> >
> > [1]:
> > https://pubs.opengroup.org/onlinepubs/9699919799/functions/mkdir.html#tag_16_325_05
>
> That's where
Am Sun, Oct 29, 2023 at 06:00:18PM +0100 schrieb Страхиња Радић:
> mkdir returning EISDIR is definitely not a part of POSIX.[1]
>
> [1]:
> https://pubs.opengroup.org/onlinepubs/9699919799/functions/mkdir.html#tag_16_325_05
That's where you're wrong. Any function can fail for any reason unless
On 23/10/29 07:59AM, Randy Palamar wrote:
> > Plus I didn't know mkdir could fail with EISDIR. My manpage is not
> > documenting that case.
>
> It's probably legacy nonsense or from some obscure platform that I
> noticed when checking other implementations [0].
mkdir returning EISDIR is
> That is one of the jobs of the -p switch to mkdir. However, without -p,
> mkdir is not allowed to ignore failure like that.
Once I looked in more detail I realized that was the case so I didn't
send the patch. It was not how I previously thought about the -p
switch though.
> Plus I didn't know
On Sun, Oct 29, 2023 at 10:58:26AM +0100, Quentin Rameau wrote:
> I pushed a patch to hackers@, you can test it and confirm that fixes
> the issue, thanks!
Works now, thanks!
Hi Rene,
> I can't find the patch in the mail archives, but git://git.suckless.org/sbase
> has commit ddde8021 (Simplify install/uninstall). With this commit
> trying to install gives the following result:
>
> % make install
> scripts/mkproto install /usr/local /usr/local/share/man proto
>
Am Sat, Oct 28, 2023 at 01:48:07PM -0600 schrieb Randy Palamar:
> This is a problem with mkdir in sbase. It probably shouldn't error out
> when mkdir(3p) fails and sets errno to EEXIST or EISDIR. I'll send a
> patch to hackers@ soon.
>
> - Randy
>
That is one of the jobs of the -p switch to
> scripts/mkproto: 15: cannot create /usr/local/share/man: Is a directory
Actually, looking at it again this is just a problem in the makefile.
scripts/mkproto has too many args. Though fixing that exposes a
separate issue with the way `find` is being invoked.
- Randy
Hi,
> scripts/mkproto: 15: cannot create /usr/local/share/man: Is a directory
This is a problem with mkdir in sbase. It probably shouldn't error out
when mkdir(3p) fails and sets errno to EEXIST or EISDIR. I'll send a
patch to hackers@ soon.
- Randy
I can't find the patch in the mail archives, but git://git.suckless.org/sbase
has commit ddde8021 (Simplify install/uninstall). With this commit
trying to install gives the following result:
% make install
scripts/mkproto install /usr/local /usr/local/share/man proto
scripts/mkproto: 15: cannot
10 matches
Mail list logo
