{"thread":{"id":"1265","subject":"[PATCH 2/5] cogito: remove use of cp -a, a non-portable GNU extension","startedAt":"2005-07-16T23:23:14Z","lastAt":"2005-07-16T23:24:10Z","messageCount":6,"participants":["Bryan Larsen"],"isPatch":true,"patchVersion":1,"patchTotal":5},"messages":[{"id":"6202","messageId":"20050716232308.7317.56890.sendpatchset@bryan-larsens-ibook-g4.local","threadId":"1265","inReplyTo":null,"subject":"[PATCH 0/5] cogito: compatibility with OS X","fromName":"Bryan Larsen","fromEmail":"bryanlarsen@yahoo.com","sentAt":"2005-07-16T23:23:14Z","receivedAt":"2005-07-16T23:23:14Z","isPatch":true,"sender":{"key":"bryan@larsen.st","avatar":"https://avatars.githubusercontent.com/u/32073?v=4"},"body":"Since Linus and Junio were unhappy with my previous set of patches to\nmake git and cogito compatible with OS X and other BSD unices, I have\nredone my patches to cogito along a similar line.\n\nThere is still one outstanding incompatibility: cogito makes use of\ncp -u. I have this message from Junio:\n\n> I do not speak for Pasky, but to me \"cp -u\" sounds just like an\n> optimization, so maybe defining CP_U='cp -u' and detect missing\n> support at config time and falling back on the simple \"cp\" would\n> be an option?\n\nPasky, does this seem reasonable? Is there a better way? So far I\nhave been able to avoid adding the \"config.sh\" contained in my first\nscript.\n\ncheers,\nBryan\n"},{"id":"6201","messageId":"20050716232319.7317.23031.sendpatchset@bryan-larsens-ibook-g4.local","threadId":"1265","inReplyTo":"20050716232308.7317.56890.sendpatchset@bryan-larsens-ibook-g4.local","subject":"[PATCH 1/5] cogito: remove use of xargs -r, a non-portable GNU extension","fromName":"Bryan Larsen","fromEmail":"bryanlarsen@yahoo.com","sentAt":"2005-07-16T23:23:25Z","receivedAt":"2005-07-16T23:23:25Z","isPatch":true,"sender":{"key":"bryan@larsen.st","avatar":"https://avatars.githubusercontent.com/u/32073?v=4"},"body":"Remove usage of xargs -r, a non-portable gnu extension. \n\nSigned-off-by: Bryan Larsen \n---\n\n cg-add | 6 +++---\n cg-init | 2 +-\n 2 files changed, 4 insertions(+), 4 deletions(-)\n\ndiff --git a/cg-add b/cg-add\n--- a/cg-add\n+++ b/cg-add\n@@ -25,8 +25,6 @@ USAGE=\"cg-add [-N] FILE...\"\n \n . ${COGITO_LIB}cg-Xlib\n \n-[ \"$1\" ] || usage\n-\n infoonly=\n while optparse; do\n \tif optparse -N; then\n@@ -36,6 +34,8 @@ while optparse; do\n \tfi\n done\n \n+[ \"$ARGS\" ] || usage\n+\n TMPFILE=$(mktemp -t gitadd.XXXXXX) || exit 1\n find \"${ARGS[@]}\" -type f -print0 > $TMPFILE || {\n \tdie \"not all files exist, nothing added\"\n@@ -43,6 +43,6 @@ find \"${ARGS[@]}\" -type f -print0 > $TMP\n }\n \n cat $TMPFILE | tr '\\0' '\\n' | sed 's/^/Adding file /'\n-cat $TMPFILE | xargs -0r git-update-cache --add ${infoonly} --\n+cat $TMPFILE | xargs -0 git-update-cache --add ${infoonly} --\n \n rm $TMPFILE\ndiff --git a/cg-init b/cg-init\n--- a/cg-init\n+++ b/cg-init\n@@ -55,7 +55,7 @@ if [ \"$uri\" ]; then\n \techo \"Cloned (origin $uri available as branch \\\"origin\\\")\"\n else\n \tgit-read-tree # Seed the dircache\n-\tfind * \\( -type f -o -type l \\) -print0 | xargs -0r cg-add ${infoonly}\n+\t[ `ls` ] && find * \\( -type f -o -type l \\) -print0 | xargs -0 cg-add ${infoonly}\n \tcg-commit -C -m\"Initial commit\" -E ${infoonly}\n fi\n \n"},{"id":"6200","messageId":"20050716232330.7317.50646.sendpatchset@bryan-larsens-ibook-g4.local","threadId":"1265","inReplyTo":"20050716232308.7317.56890.sendpatchset@bryan-larsens-ibook-g4.local","subject":"[PATCH 2/5] cogito: remove use of cp -a, a non-portable GNU extension","fromName":"Bryan Larsen","fromEmail":"bryanlarsen@yahoo.com","sentAt":"2005-07-16T23:23:36Z","receivedAt":"2005-07-16T23:23:36Z","isPatch":true,"sender":{"key":"bryan@larsen.st","avatar":"https://avatars.githubusercontent.com/u/32073?v=4"},"body":"Remove usage of cp -a, a non-portable gnu extension.\n\nSigned-off-by: Bryan Larsen \n---\n\n cg-pull | 2 +-\n 1 files changed, 1 insertions(+), 1 deletions(-)\n\ndiff --git a/cg-pull b/cg-pull\n--- a/cg-pull\n+++ b/cg-pull\n@@ -217,7 +217,7 @@ fetch_local () {\n \t[ \"$1\" = \"-i\" ] && shift\n \t[ \"$1\" = \"-s\" ] && shift\n \n-\tcp_flags_l=\"-va\"\n+\tcp_flags_l=\"-vdpR\"\n \tif [ \"$1\" = \"-u\" ]; then\n \t\tcp_flags_l=\"$cp_flags_l -u$can_hardlink\"\n \t\tsuggest_hardlink=\n"},{"id":"6203","messageId":"20050716232341.7317.4962.sendpatchset@bryan-larsens-ibook-g4.local","threadId":"1265","inReplyTo":"20050716232308.7317.56890.sendpatchset@bryan-larsens-ibook-g4.local","subject":"[PATCH 3/5] cogito: try harder to find gnu stat","fromName":"Bryan Larsen","fromEmail":"bryanlarsen@yahoo.com","sentAt":"2005-07-16T23:23:55Z","receivedAt":"2005-07-16T23:23:55Z","isPatch":true,"sender":{"key":"bryan@larsen.st","avatar":"https://avatars.githubusercontent.com/u/32073?v=4"},"body":"Look harder for gnu stat.\n\nCogito has code to use awk if gnu stat is missing. Look harder \nfor gnu stat under alternate names such as gstat and gnustat, avoiding\nthe use of awk if possible.\n\nSigned-off-by: Bryan Larsen \n---\n\n cg-Xlib | 6 +++---\n 1 files changed, 3 insertions(+), 3 deletions(-)\n\ndiff --git a/cg-Xlib b/cg-Xlib\n--- a/cg-Xlib\n+++ b/cg-Xlib\n@@ -229,11 +229,11 @@ fi\n \n \n # Compatibility hacks:\n-# Fortunately none as of now.\n-\n \n export BROKEN_MKTEMP=1\n del=$($(which mktemp) -t 2>/dev/null) && { rm $del; export BROKEN_MKTEMP=; }\n-has_stat=$(which stat 2>/dev/null)\n+has_stat=$(which gnustat 2> /dev/null ||\n+ which gstat 2> /dev/null ||\n+ which stat 2>/dev/null)\n [ \"$has_stat\" ] && $has_stat -c %s / 2>/dev/null >&2 || has_stat=\n has_gnudate=$(date -Rud \"1970-01-01 UTC\" 2>/dev/null)\n"},{"id":"6205","messageId":"20050716232353.7317.63574.sendpatchset@bryan-larsens-ibook-g4.local","threadId":"1265","inReplyTo":"20050716232308.7317.56890.sendpatchset@bryan-larsens-ibook-g4.local","subject":"[PATCH 4/5] cogito: try harder to find gnu date","fromName":"Bryan Larsen","fromEmail":"bryanlarsen@yahoo.com","sentAt":"2005-07-16T23:24:02Z","receivedAt":"2005-07-16T23:24:02Z","isPatch":true,"sender":{"key":"bryan@larsen.st","avatar":"https://avatars.githubusercontent.com/u/32073?v=4"},"body":"Look harder for gnu date, use if available.\n\nSigned-off-by: Bryan Larsen \n---\n\n cg-Xlib | 8 ++++++--\n 1 files changed, 6 insertions(+), 2 deletions(-)\n\ndiff --git a/cg-Xlib b/cg-Xlib\n--- a/cg-Xlib\n+++ b/cg-Xlib\n@@ -70,7 +70,7 @@ showdate () {\n \tsecs=$(($secs + $tzhours * 3600 + $tzmins * 60))\n \t[ \"$format\" ] || format=\"+%a, %d %b %Y %H:%M:%S $2\"\n \tif [ \"$has_gnudate\" ]; then\n-\t\tLANG=C date -ud \"1970-01-01 UTC + $secs sec\" \"$format\"\n+\t\tLANG=C ${has_gnudate} -ud \"1970-01-01 UTC + $secs sec\" \"$format\"\n \telse\n \t\tLANG=C date -u -r $secs \"$format\"\n \tfi\n@@ -236,4 +236,8 @@ has_stat=$(which gnustat 2> /dev/null ||\n which gstat 2> /dev/null ||\n which stat 2>/dev/null)\n [ \"$has_stat\" ] && $has_stat -c %s / 2>/dev/null >&2 || has_stat=\n-has_gnudate=$(date -Rud \"1970-01-01 UTC\" 2>/dev/null)\n+has_gnudate=$(which gnudate 2> /dev/null || \n+ which gdate 2> /dev/null || \n+ which stat 2>/dev/null)\n+[ \"$has_gnudate\" ] && \n+ $has_gnudate -Rud \"1970-01-01 UTC\" 2> /dev/null >&2 || has_gnudate=\n"},{"id":"6204","messageId":"20050716232404.7317.86971.sendpatchset@bryan-larsens-ibook-g4.local","threadId":"1265","inReplyTo":"20050716232308.7317.56890.sendpatchset@bryan-larsens-ibook-g4.local","subject":"[PATCH 5/5] cogito: remove findutils dependency from Portfile","fromName":"Bryan Larsen","fromEmail":"bryanlarsen@yahoo.com","sentAt":"2005-07-16T23:24:10Z","receivedAt":"2005-07-16T23:24:10Z","isPatch":true,"sender":{"key":"bryan@larsen.st","avatar":"https://avatars.githubusercontent.com/u/32073?v=4"},"body":"Gnu findutils (xargs) is no longer required; remove the dependency. \nGnu coreutils is still required, but only if awk is not installed.\n\nSigned-off-by: Bryan Larsen \n---\n\n Portfile.in | 3 +--\n 1 files changed, 1 insertions(+), 2 deletions(-)\n\ndiff --git a/Portfile.in b/Portfile.in\n--- a/Portfile.in\n+++ b/Portfile.in\n@@ -14,8 +14,7 @@ long_description The git core, develope\n homepage http://kernel.org/pub/software/scm/cogito/\n master_sites http://kernel.org/pub/software/scm/cogito/\n configure\t {}\n-depends_lib bin:gcp:coreutils\n-depends_lib bin:gnuxargs:findutils\n+depends_lib bin:gstat:coreutils\n build.type gnu\n destroot.type gnu\n destroot.destdir prefix=${prefix} DESTDIR=${destroot}\n"}]}