cvs commit: ports/astro/gpsdrive pkg-plist ports/audio/gqmpeg-devel pkg-plist ports/databases/pgworksheet pkg-plist ports/deskutils/hot-babe pkg-plist ports/deskutils/xchm pkg-plist ports/editors/abiword-devel pkg-plist ports/games/crimson ...

Dejan Lesjak dejan.lesjak at ijs.si
Wed Jun 15 11:52:53 GMT 2005


On Wednesday 15 of June 2005 13:10, Alexander Leidinger wrote:
> Dejan Lesjak <dejan.lesjak at ijs.si> wrote:
> > On Wednesday 15 of June 2005 12:07, Alexander Leidinger wrote:
> >> Michael Johnson <ahze at ahze.net> wrote:
> >> > On Jun 14, 2005, at 10:43 PM, Dejan Lesjak wrote:
> >> >> lesi        2005-06-15 02:43:36 UTC
> >> >>
> >> >>   FreeBSD ports repository
> >> >>
> >> >>   Log:
> >> >>   Directory share/pixmaps is now included in mtree
> >> >>   (BSD.x11-4.dist rev. 1.27), so remove it from plist.
> >> >
> >> > sweet!
> >>
> >> We need to bump the revision number of those ports, else no updated
> >> packages will be generated.
> >
> > The packages should include mtree that is present when they are generated
> > (at least as far as I understand this). So there's no need to generate
> > packages that only move a directory from +MTREE_DIRS to +CONTENTS.
>
> I was talking about the case where a directory moved from +CONTENTS to the
> mtree file. We don't want to remove a mtree directory in such a port, don't
> we?

Well, if they are empty, why not? It is actually rather unavoidable. I could 
be missing something, but this is how I understand things:
- packages made before this change will have share/pixmap in +CONTENTS, so 
this directory will get removed if it is empty and there is nothing we can 
now do about that, since those packages with their +CONTENTS are already 
installed on users machines. Nothing however should break because of this 
since (1) packages that would rely on this directory being present already 
have it listed either in +CONTENTS or +MTREE_DIRS and (2) things outside of 
packages that would rely on this directory present would already be broken 
before since these packages removed share/pixmaps directory if empty already.
- packages made after this change will have share/pixmaps in +MTREE_DIRS, so 
it won't get removed upon deinstall of such packages. This shouldn't be a 
problem as far as I can see.


Dejan


More information about the cvs-all mailing list