Thorsten Glaser | 2 Feb 2007 00:48
Picon
Gravatar

Re: CVS: herc.mirbsd.org: ports

Thorsten Glaser dixit:

>Log message:
>bring in bsiegert <at> 's "wtf don't eat my fscking log messages!" patch

you missed a free commit ;)

//mirabile
--

-- 
  "Using Lynx is like wearing a really good pair of shades: cuts out
   the glare and harmful UV (ultra-vanity), and you feel so-o-o COOL."
                                         -- Henry Nelson, March 1999

Thorsten Glaser | 2 Feb 2007 00:47
Picon
Gravatar

Re: CVS: herc.mirbsd.org: src

Thorsten Glaser dixit:

>Commit ID:	10045C27A5403C6060F
[…]
>the standards say these two are undefined, so why shouldn't we
>be so liberal and define them to something that actually makes
>sense? (I could even implement an unget buffer so that all but
>the first octet are eaten, but that'd slow down I guess.) Code
>in the base system is hereby allowed to use this, and all fol-
>lowing, enhancements to the standards, because MirOS is a core
>oriented "comes as a package" OS and we do not need to require
>our own tools to only use normed functions unless it's made to
>a portable package.

This is the first of a series of commits (I think) I plan to be
using the "Plan 9 way" of doing I18N. Existing APIs suck, so we
implement them for third-party software, but use our own better
APIs (specifically, I plan to implement most of Plan 9's APIs &
a few own, e.g. to go back in a MB string to the last character
or to the last column - for e.g. col(1) useful).

I even plan to drop __locale_is_utf8 and make the 'C' locale an
equivalent of the 'UTF-8' locale. This would simplify the code,
speed it up, and the 'C' locale is 7-bit anyway since we do not
support latin1 any more in recent snapshots (and nobody sent me
a complaint yet).

Benny, what do you think? Anyone else?

bye,
(Continue reading)

Thorsten Glaser | 2 Feb 2007 00:58
Picon
Gravatar

pkgtools: <at> exec _still_ broken

tg <at> herc:~/tmp $ sudo pkg_delete -c php-core
pkg_delete(php-core): attempting to delete non-existent directory '/usr/mpkg/lib/php/modules'
this packing list is incorrect - ignoring delete request

$ tail -5 /usr/ports/www/php/core/pkg/PLIST                                             
 <at> dirrm share/doc/php
 <at> comment Always add  <at> dirrm lib/php/modules
 <at> dirrm lib/php/modules
 <at> exec mkdir -p lib/php/modules
 <at> dirrm lib/php

I suppose 'lib/php/modules' is created at pkg_add time in the
playground, but not transferred over. I wonder if we want to
run  <at> exec in the playground or in the final destination; in
the first case, 'update-plist.pl' should probably add it as
a directory target (foo/)… oops, we don't have them… ok, so
how about every  <at> dirrm is statted _after_ copying the play-
ground into the localbase, and, if it doesn't exist, it's
mkdir -p'd. (We need a mkdir_p() function in libc.)

bye,
//mirabile
--

-- 
  "Using Lynx is like wearing a really good pair of shades: cuts out
   the glare and harmful UV (ultra-vanity), and you feel so-o-o COOL."
                                         -- Henry Nelson, March 1999

Thorsten Glaser | 2 Feb 2007 01:09
Picon
Gravatar

Re: pkgtools: <at> exec _still_ broken

Dixi:

>how about every  <at> dirrm is statted _after_ copying the play-
>ground into the localbase, and, if it doesn't exist, it's
>mkdir -p'd. (We need a mkdir_p() function in libc.)

That other BSD has:
 <at> chmod 0700
 <at> owner ...
 <at> group ...
 <at> sample ${DB_DIR}

Sounds reasonable?

//mirabile
--

-- 
  "Using Lynx is like wearing a really good pair of shades: cuts out
   the glare and harmful UV (ultra-vanity), and you feel so-o-o COOL."
                                         -- Henry Nelson, March 1999

Benny Siegert | 2 Feb 2007 08:56
Picon
Gravatar

Re: pkgtools: <at> exec _still_ broken

>> how about every  <at> dirrm is statted _after_ copying the play-
>> ground into the localbase, and, if it doesn't exist, it's
>> mkdir -p'd. (We need a mkdir_p() function in libc.)
>
> That other BSD has:
>  <at> chmod 0700
>  <at> owner ...
>  <at> group ...
>  <at> sample ${DB_DIR}
>
> Sounds reasonable?

Yes and no. Only for directories _outside_ LOCALBASE.

--Benny.

Benny Siegert | 2 Feb 2007 08:55
Picon
Gravatar

Re: pkgtools: <at> exec _still_ broken

> in
> the first case, 'update-plist.pl' should probably add it as
> a directory target (foo/)… oops, we don't have them…

Yes, we do. And if we switched to foo/ after I implemented it, we  
would not have this problem. But for some reason, you did not want to.

> ok, so
> how about every  <at> dirrm is statted _after_ copying the play-
> ground into the localbase, and, if it doesn't exist, it's
> mkdir -p'd. (We need a mkdir_p() function in libc.)

No.

Thorsten Glaser | 2 Feb 2007 09:20
Picon
Gravatar

Re: pkgtools: <at> exec _still_ broken

Benny Siegert dixit:

>> ok, so
>> how about every  <at> dirrm is statted _after_ copying the play-
>> ground into the localbase, and, if it doesn't exist, it's
>> mkdir -p'd. (We need a mkdir_p() function in libc.)
>
> No.

[…]
>> Sounds reasonable?
>
> Yes and no. Only for directories _outside_ LOCALBASE.

And what do we then? (Why is  <at> exec not run within the localbase?)

//mirabile
--

-- 
  "Using Lynx is like wearing a really good pair of shades: cuts out
   the glare and harmful UV (ultra-vanity), and you feel so-o-o COOL."
                                         -- Henry Nelson, March 1999

Benny Siegert | 2 Feb 2007 12:47
Picon
Gravatar

Re: pkgtools: <at> exec _still_ broken

>>> Sounds reasonable?
>>
>> Yes and no. Only for directories _outside_ LOCALBASE.
>
> And what do we then?

The two possibilities are:
a) use the dir/ form (for ALL directories in the plist) -- you will  
have to do it manuallly since our update-plist does not grok dir/
b) Simply prefix the directory to create with %D in the plist (i.e.  
 <at> exec mkdir -p %D/foo/bar). No, I won't hack code into pkgtools to  
make it change all mkdir commands, update-plist is the right place to  
do it.

> (Why is  <at> exec not run within the localbase?)

I think this is actually undocumented but it has always been that  
way. That's why you should prefix _everything_, including "relative"  
paths with %D. In short, you should not depend on where  <at> exec is run.

--Benny.

Thorsten Glaser | 2 Feb 2007 12:59
Picon
Gravatar

Re: pkgtools: <at> exec _still_ broken

Benny Siegert dixit:

> I think this is actually undocumented but it has always been that way. That's
> why you should prefix _everything_, including "relative" paths with %D. In
> short, you should not depend on where  <at> exec is run.

Ok, I'll fix update-plist, you'll document it ;)

//mirabile
--

-- 
  "Using Lynx is like wearing a really good pair of shades: cuts out
   the glare and harmful UV (ultra-vanity), and you feel so-o-o COOL."
                                         -- Henry Nelson, March 1999


Gmane