From mboxrd@z Thu Jan 1 00:00:00 1970 From: David Aguilar Subject: [PATCH v6 3/3] diff: generate pretty filenames in prep_temp_blob() Date: Sat, 30 May 2009 21:37:55 -0700 Message-ID: <1243744675-24160-3-git-send-email-davvid@gmail.com> References: <1243744675-24160-1-git-send-email-davvid@gmail.com> <1243744675-24160-2-git-send-email-davvid@gmail.com> Cc: gitster@pobox.com, peff@peff.net, markus.heidelberg@web.de, jnareb@gmail.com, j.sixt@viscovery.net, David Aguilar To: git@vger.kernel.org X-From: git-owner@vger.kernel.org Sun May 31 06:38:25 2009 Return-path: Envelope-to: gcvg-git-2@gmane.org Received: from vger.kernel.org ([209.132.176.167]) by lo.gmane.org with esmtp (Exim 4.50) id 1MAcoR-0008Bb-M1 for gcvg-git-2@gmane.org; Sun, 31 May 2009 06:38:24 +0200 Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751264AbZEaEiK (ORCPT ); Sun, 31 May 2009 00:38:10 -0400 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1751149AbZEaEiI (ORCPT ); Sun, 31 May 2009 00:38:08 -0400 Received: from mail-pz0-f177.google.com ([209.85.222.177]:57197 "EHLO mail-pz0-f177.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751214AbZEaEiG (ORCPT ); Sun, 31 May 2009 00:38:06 -0400 Received: by mail-pz0-f177.google.com with SMTP id 7so5426611pzk.33 for ; Sat, 30 May 2009 21:38:08 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=gamma; h=domainkey-signature:received:received:from:to:cc:subject:date :message-id:x-mailer:in-reply-to:references; bh=idLn8cOK1pHCozJNE3ZcpLZQUVTnhQyLbx66/LGixAw=; b=oBSuQwwa8veI4+jY56UacOQSAF7CHuOOBQlgMnZEZcp0qNIyuwsVGOVbVJtCYGUiyL kPZYRIzr71aVBSsBmnFgO3W1G7thMFRNUrbM46GXehbvWI9F81oium2SWqz3PzhrVIsj yQ0Gz1g8ZsQxMkNmajEu/IEewOthHfcWsEbqw= DomainKey-Signature: a=rsa-sha1; c=nofws; d=gmail.com; s=gamma; h=from:to:cc:subject:date:message-id:x-mailer:in-reply-to:references; b=ue3hyk6gX31e+awhV3+ZB0u7qGLSL5jOU5pK8Gjz4r7Fniwn0nydibU+XEVg1Ul8PP 8uK9barjXwO2JaojouZYFt2HDHl2yvqiXXcS7Kx6d45Urv2jfm6j14ul2CSwH9IgGskD uEVzguytz5tMfKTh4q6bJE3PNVP/ShAPTu5vk= Received: by 10.141.179.5 with SMTP id g5mr3991491rvp.144.1243744688569; Sat, 30 May 2009 21:38:08 -0700 (PDT) Received: from localhost (208-106-56-2.static.dsltransport.net [208.106.56.2]) by mx.google.com with ESMTPS id k37sm4170329waf.42.2009.05.30.21.38.07 (version=TLSv1/SSLv3 cipher=RC4-MD5); Sat, 30 May 2009 21:38:08 -0700 (PDT) X-Mailer: git-send-email 1.6.3.1.178.g5b6b2 In-Reply-To: <1243744675-24160-2-git-send-email-davvid@gmail.com> Sender: git-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: git@vger.kernel.org Archived-At: Naturally, prep_temp_blob() did not care about filenames. As a result, GIT_EXTERNAL_DIFF and textconv generated filenames such as ".diff_XXXXXX". This modifies prep_temp_blob() to generate user-friendly filenames when creating temporary files. Diffing "name.ext" now generates "XXXXXX_name.ext". Signed-off-by: David Aguilar --- cache.h | 2 ++ diff.c | 12 +++++++++++- path.c | 16 ++++++++++++++++ t/t4020-diff-external.sh | 9 +++++++++ 4 files changed, 38 insertions(+), 1 deletions(-) diff --git a/cache.h b/cache.h index b8503ad..871c984 100644 --- a/cache.h +++ b/cache.h @@ -614,6 +614,8 @@ extern int is_empty_blob_sha1(const unsigned char *sha1); int git_mkstemp(char *path, size_t n, const char *template); +int git_mkstemps(char *path, size_t n, const char *template, int suffix_len); + /* * NOTE NOTE NOTE!! * diff --git a/diff.c b/diff.c index dcfbcb0..4d0a5b9 100644 --- a/diff.c +++ b/diff.c @@ -1964,8 +1964,16 @@ static void prep_temp_blob(const char *path, struct diff_tempfile *temp, { int fd; struct strbuf buf = STRBUF_INIT; + struct strbuf template = STRBUF_INIT; + char *path_dup = xstrdup(path); + const char *base = basename(path_dup); - fd = git_mkstemp(temp->tmp_path, PATH_MAX, ".diff_XXXXXX"); + /* Generate "XXXXXX_basename.ext" */ + strbuf_addstr(&template, "XXXXXX_"); + strbuf_addstr(&template, base); + + fd = git_mkstemps(temp->tmp_path, PATH_MAX, template.buf, + strlen(base) + 1); if (fd < 0) die("unable to create temp-file: %s", strerror(errno)); if (convert_to_working_tree(path, @@ -1981,6 +1989,8 @@ static void prep_temp_blob(const char *path, struct diff_tempfile *temp, temp->hex[40] = 0; sprintf(temp->mode, "%06o", mode); strbuf_release(&buf); + strbuf_release(&template); + free(path_dup); } static struct diff_tempfile *prepare_temp_file(const char *name, diff --git a/path.c b/path.c index 8a0a674..047fdb0 100644 --- a/path.c +++ b/path.c @@ -139,6 +139,22 @@ int git_mkstemp(char *path, size_t len, const char *template) return mkstemp(path); } +/* git_mkstemps() - create tmp file with suffix honoring TMPDIR variable. */ +int git_mkstemps(char *path, size_t len, const char *template, int suffix_len) +{ + const char *tmp; + size_t n; + + tmp = getenv("TMPDIR"); + if (!tmp) + tmp = "/tmp"; + n = snprintf(path, len, "%s/%s", tmp, template); + if (len <= n) { + errno = ENAMETOOLONG; + return -1; + } + return mkstemps(path, suffix_len); +} int validate_headref(const char *path) { diff --git a/t/t4020-diff-external.sh b/t/t4020-diff-external.sh index 0720001..4ea42e0 100755 --- a/t/t4020-diff-external.sh +++ b/t/t4020-diff-external.sh @@ -136,6 +136,15 @@ test_expect_success 'GIT_EXTERNAL_DIFF with more than one changed files' ' GIT_EXTERNAL_DIFF=echo git diff ' +test_expect_success 'GIT_EXTERNAL_DIFF generates pretty paths' ' + touch file.ext && + git add file.ext && + echo with extension > file.ext && + GIT_EXTERNAL_DIFF=echo git diff file.ext | grep ......_file\.ext && + git update-index --force-remove file.ext && + rm file.ext +' + echo "#!$SHELL_PATH" >fake-diff.sh cat >> fake-diff.sh <<\EOF cat $2 >> crlfed.txt -- 1.6.3.1.178.g5b6b2