Skip site navigation (1)Skip section navigation (2)
Date:      Tue, 26 Aug 2008 23:18:50 +0200
From:      Anders Nore <andenore@freebsd.org>
To:        Garrett Cooper <yanefbsd@gmail.com>
Cc:        "freebsd-hackers@freebsd.org" <freebsd-hackers@freebsd.org>
Subject:   Re: pkg_improved GSoC 2008 finished
Message-ID:  <op.ugiitopep8qiwu@duckjen.nextgentel.no>
In-Reply-To: <7d6fde3d0808222236t45103e4eseb8f174ffb89fdca@mail.gmail.com>
References:  <op.uf648rn7p8qiwu@duckjen.nextgentel.no> <1d3ed48c0808211456h779bec94n9b6597c66f7741b2@mail.gmail.com> <1d3ed48c0808211500i4445b443t4a67da7cf514adaa@mail.gmail.com> <op.uf98j4jrp8qiwu@duckjen.nextgentel.no> <7d6fde3d0808222236t45103e4eseb8f174ffb89fdca@mail.gmail.com>

next in thread | previous in thread | raw e-mail | index | archive | help
On Sat, 23 Aug 2008 07:36:02 +0200, Garrett Cooper <yanefbsd@gmail.com>  
wrote:
>
> How about this?
>
> For standalone packages, specify:
>
> Makefile.inc <-- redefine variables here.
> pkg_install/  <-- sources go here.
>
> Seems like a simple solution ;).
>
> As for diff, you probably what to use:
>
> --exclude-from=SCM_patterns.txt
>
> where SCM_patterns.txt consists of:
> --------------
> CVS
> .git
> .svn
> .CC
> --------------
> ... etc.
>
> As for patch, `patch -p0 < pkg_install.patch' does the trick if you do
> `diff -Naur .'.
>
> Cheers and good work on finishing off this awesome task. I'll gladly
> write up tests for you if you like =],
> -Garrett


I've been making a lot of mistakes trying to make correct patches, but I  
think I finally got a hang of it now :)
I'd really appreciate some tests if you're willing to write some.

I've also discovered two bugs in all -CURRENT, -RELENG_7 and -RELENG_7_0  
of pkg_install:

1. Bug in pkg_info -W, it does not handle empty @cwd's correctly in the  
packinglist, so if you have any packages with an empty @cwd before the  
files listed it will not show as installed by a package.
e.g., if you have the kdeadmin-4.1.0 package installed, one of the files  
it installs is `/usr/local/kde4/libdata/ldconfig/kdeadmin4' but running  
`pkg_info -W /usr/local/kde4/libdata/ldconfig/kdeadmin4' will not show  
that it installed by kdeadmin-4.1.0.

2. Bug in pkg_add, adding two packages where the first package recursively  
pkg_add's dependencies using slave mode will cause an `pkg_add: fatal  
error during execution: getcwd'. Because it doesn't save and recover  
previous working directories correctly.
e.g., if you delete packages "joe-3.5,1", "gettext-0.16.1_3" and  
"zip-2.32", and run `pkg_add -r joe zip' this error will occur as joe  
depends on gettext. If you run `pkg_add -r zip joe' this error will not  
occur.

Here's a patch for these errors:  
http://home.no.net/andenore/patches/pkg_install_bugfix-2008-08-26_RELENG_7.diff
Can someone confirm these bugs, and if the patch is suitable?


Thanks a bunch,
Anders Nore



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?op.ugiitopep8qiwu>