Hi, I keep my old laptop by installing packages manually instead of building it, I ear about pkg-get but in the practice I can do whatever it does by using prt-get and configuring PKGMK_PACKAGE_DIR variable in pkgmk.conf. However after playing a bit, see that pkgmk does something odd: bash-3.2# cd /usr/ports/core/autoconf bash-3.2# grep PKGMK_PACKAGE_DIR /etc/pkgmk.conf PKGMK_PACKAGE_DIR="/usr/pkg" bash-3.2# pkgmk -utd -cf /etc/pkgmk.conf =======> Package '/usr/pkg/autoconf#2.62-1.pkg.tar.gz' is not up to date. bash-3.2# file /usr/pkg/pkg-get#0.4.5-2.pkg.tar.gz /usr/pkg/pkg-get#0.4.5-2.pkg.tar.gz: gzip compressed data, from Unix, last modified: Sat Jun 7 00:46:33 2008 bash-3.2# bash -x /usr/bin/pkgmk -utd -cf /etc/pkgmk.conf 2>&1 | tail -10 ++ '[' '!' -e Pkgfile ']' ++ '[' '!' /usr/pkg/autoconf#2.62-1.pkg.tar.gz -nt Pkgfile ']' ++ RESULT=yes ++ break ++ echo yes + '[' yes = yes ']' + info 'Package '\''/usr/pkg/autoconf#2.62-1.pkg.tar.gz'\'' is not up to date.' + echo '=======> Package '\''/usr/pkg/autoconf#2.62-1.pkg.tar.gz'\'' is not up to date.' =======> Package '/usr/pkg/autoconf#2.62-1.pkg.tar.gz' is not up to date. + exit 0 It tries to check if the package is newer than the Pkgfile (test FILE1 -nt FILE2), so it breaks and returns yes as result for `build_needed`. Here is a piece of `build_needed` function: [...] RESULT="no" for FILE in $PKGMK_PKGFILE ${source[@]}; do FILE=`get_filename $FILE` if [ ! -e $FILE ] || [ ! $TARGET -nt $FILE ]; then RESULT="yes" break [...] After 'touch' the package it should do the trick, but also the problem appears and seems that I need all sources for checking if package is up to date. IMO it shouldn't be the default action to take, and maybe by would be better a switch option. I attached a patch with my idea. I prefer to comment this improvement before reporting it in FS, what are your opinions? Best regards, Jose V Beneyto diff --git a/scripts/pkgmk.in b/scripts/pkgmk.in index 9b193bb..42b7f11 100755 --- a/scripts/pkgmk.in +++ b/scripts/pkgmk.in @@ -466,13 +466,16 @@ build_needed() { RESULT="yes" if [ -f $TARGET ]; then RESULT="no" - for FILE in $PKGMK_PKGFILE ${source[@]}; do - FILE=`get_filename $FILE` - if [ ! -e $FILE ] || [ ! $TARGET -nt $FILE ]; then - RESULT="yes" - break - fi - done + [ ! $TARGET -nt $PKGMK_PKGFILE ] && RESULT="yes" + if [ "$PKGMK_CHECK_SOURCES" != "no" ]; then + for FILE in ${source[@]}; do + FILE=`get_filename $FILE` + if [ ! -e $FILE ] || [ ! $TARGET -nt $FILE ]; then + RESULT="yes" + break + fi + done + fi fi echo $RESULT @@ -503,6 +506,7 @@ print_help() { echo " -um, --update-md5sum update md5sum" echo " -im, --ignore-md5sum build package without checking md5sum" echo " -cm, --check-md5sum do not build, only check md5sum" + echo " -cs, --check-sources do not build without having sources" echo " -ns, --no-strip do not strip executable binaries or libraries" echo " -f, --force build package even if it appears to be up to date" echo " -c, --clean remove package and downloaded files" @@ -538,6 +542,8 @@ parse_options() { PKGMK_IGNORE_MD5SUM="yes" ;; -cm|--check-md5sum) PKGMK_CHECK_MD5SUM="yes" ;; + -cs|--check-sources) + PKGMK_CHECK_SOURCES="yes" ;; -ns|--no-strip) PKGMK_NO_STRIP="yes" ;; -f|--force) @@ -669,6 +675,7 @@ PKGMK_KEEP_WORK="no" PKGMK_UPDATE_MD5SUM="no" PKGMK_IGNORE_MD5SUM="no" PKGMK_CHECK_MD5SUM="no" +PKGMK_CHECK_SOURCES="no" PKGMK_NO_STRIP="no" PKGMK_CLEAN="no"