{"thread":{"id":"41711","subject":"Forgotten declaration of function path_name() in revision.h?","startedAt":"2016-03-16T18:05:49Z","lastAt":"2016-03-16T19:08:50Z","messageCount":4,"participants":["Petr Stodulka","Jeff King","Eric Sunshine"],"isPatch":false,"patchVersion":null,"patchTotal":null},"messages":[{"id":"280935","messageId":"56E9A07D.3080508@redhat.com","threadId":"41711","inReplyTo":null,"subject":"Forgotten declaration of function path_name() in revision.h?","fromName":"Petr Stodulka","fromEmail":"pstodulk@redhat.com","sentAt":"2016-03-16T18:05:49Z","receivedAt":"2016-03-16T18:05:49Z","isPatch":false,"sender":{"key":"pstodulk@redhat.com","avatar":"https://avatars.githubusercontent.com/u/1656374?v=4"},"body":"Hi,\naccording to commit 9831e92 (merge) there is maybe by mistake kept declaration\nof function path_name() in revision.h, whose definition was removed\nand isn't used in git anymore.\n\nRegards,\nPetr\n\n\n======================================================================\nFrom ae72c8f9085b3b7bd84f94f90ff5b0416db59d67 Mon Sep 17 00:00:00 2001\nFrom: Petr Stodulka \nDate: Wed, 16 Mar 2016 18:51:53 +0100\nSubject: [PATCH] remove obsoleted function path_name() from header file\n revision.h\n\nThe function was removed by commit 9831e92 due to possible security\nissue.\n---\n revision.h | 2 --\n 1 file changed, 2 deletions(-)\n\ndiff --git a/revision.h b/revision.h\nindex dca0d38..f20168e 100644\n--- a/revision.h\n+++ b/revision.h\n@@ -257,8 +257,6 @@ extern void put_revision_mark(const struct rev_info *revs,\n extern void mark_parents_uninteresting(struct commit *commit);\n extern void mark_tree_uninteresting(struct tree *tree);\n \n-char *path_name(struct strbuf *path, const char *name);\n-\n extern void show_object_with_name(FILE *, struct object *, const char *);\n \n extern void add_pending_object(struct rev_info *revs,\n-- \n2.4.3\n\n"},{"id":"280943","messageId":"20160316184853.GA32615@sigill.intra.peff.net","threadId":"41711","inReplyTo":"56E9A07D.3080508@redhat.com","subject":"Re: Forgotten declaration of function path_name() in revision.h?","fromName":"Jeff King","fromEmail":"peff@peff.net","sentAt":"2016-03-16T18:48:53Z","receivedAt":"2016-03-16T18:48:53Z","isPatch":false,"sender":{"key":"peff@peff.net","avatar":"https://avatars.githubusercontent.com/u/45925?v=4"},"body":"On Wed, Mar 16, 2016 at 07:05:49PM +0100, Petr Stodulka wrote:\n\n> according to commit 9831e92 (merge) there is maybe by mistake kept declaration\n> of function path_name() in revision.h, whose definition was removed\n> and isn't used in git anymore.\n\nYes, this should have been part of de1e67d (list-objects: pass full\npathname to callbacks, 2016-02-11), but I missed it.\n\nThe patch itself looks fine to me (though it probably makes more sense\nto point to de1e67d than the merge).\n\nNote also that:\n\n> ======================================================================\n> From ae72c8f9085b3b7bd84f94f90ff5b0416db59d67 Mon Sep 17 00:00:00 2001\n> From: Petr Stodulka \n> Date: Wed, 16 Mar 2016 18:51:53 +0100\n> Subject: [PATCH] remove obsoleted function path_name() from header file\n> revision.h\n\nIt makes things easier on the maintainer if you format your patch such\nthat \"git am\" can apply it directly. Use scissors like:\n\n-- >8 --\n\nto separate the patch from anything that should not go into the commit\nmessage (rather than \"=====...\" as you have here).\n\nDrop the \"From \" line which says nothing (it is an mbox separator, but\nwe are already inside a message).\n\nThe rest of the headers do not hurt, but are generally redundant with\nwhat is in your email header (though in this case, the Subject is\ndifferent, so you would want to retain that).\n\n-Peff\n"},{"id":"280944","messageId":"CAPig+cReEC1swN8oyD68SqHAJhV2i1_CGEO25sJfA-4gA7jP0w@mail.gmail.com","threadId":"41711","inReplyTo":"20160316184853.GA32615@sigill.intra.peff.net","subject":"Re: Forgotten declaration of function path_name() in revision.h?","fromName":"Eric Sunshine","fromEmail":"sunshine@sunshineco.com","sentAt":"2016-03-16T18:52:03Z","receivedAt":"2016-03-16T18:52:03Z","isPatch":false,"sender":{"key":"sunshine@sunshineco.com","avatar":"https://avatars.githubusercontent.com/u/163641?v=4"},"body":"On Wed, Mar 16, 2016 at 2:48 PM, Jeff King wrote:\n> It makes things easier on the maintainer if you format your patch such\n> that \"git am\" can apply it directly. Use scissors like:\n>\n> -- >8 --\n>\n> to separate the patch from anything that should not go into the commit\n> message (rather than \"=====...\" as you have here).\n>\n> Drop the \"From \" line which says nothing (it is an mbox separator, but\n> we are already inside a message).\n>\n> The rest of the headers do not hurt, but are generally redundant with\n> what is in your email header (though in this case, the Subject is\n> different, so you would want to retain that).\n\nAlso, Signed-off-by: is missing.\n"},{"id":"280945","messageId":"56E9AF42.1010907@redhat.com","threadId":"41711","inReplyTo":"20160316184853.GA32615@sigill.intra.peff.net","subject":"Re: Forgotten declaration of function path_name() in revision.h?","fromName":"Petr Stodulka","fromEmail":"pstodulk@redhat.com","sentAt":"2016-03-16T19:08:50Z","receivedAt":"2016-03-16T19:08:50Z","isPatch":false,"sender":{"key":"pstodulk@redhat.com","avatar":"https://avatars.githubusercontent.com/u/1656374?v=4"},"body":"On 16.3.2016 19:48, Jeff King wrote:\n> On Wed, Mar 16, 2016 at 07:05:49PM +0100, Petr Stodulka wrote:\n> \n>> according to commit 9831e92 (merge) there is maybe by mistake kept declaration\n>> of function path_name() in revision.h, whose definition was removed\n>> and isn't used in git anymore.\n> \n> Yes, this should have been part of de1e67d (list-objects: pass full\n> pathname to callbacks, 2016-02-11), but I missed it.\n> \n> The patch itself looks fine to me (though it probably makes more sense\n> to point to de1e67d than the merge).\n> \n> Note also that:\n> \n>> ======================================================================\n>> From ae72c8f9085b3b7bd84f94f90ff5b0416db59d67 Mon Sep 17 00:00:00 2001\n>> From: Petr Stodulka \n>> Date: Wed, 16 Mar 2016 18:51:53 +0100\n>> Subject: [PATCH] remove obsoleted function path_name() from header file\n>> revision.h\n> \n> It makes things easier on the maintainer if you format your patch such\n> that \"git am\" can apply it directly. Use scissors like:\n> \n> -- >8 --\n> \n> to separate the patch from anything that should not go into the commit\n> message (rather than \"=====...\" as you have here).\n> \n> Drop the \"From \" line which says nothing (it is an mbox separator, but\n> we are already inside a message).\n> \n> The rest of the headers do not hurt, but are generally redundant with\n> what is in your email header (though in this case, the Subject is\n> different, so you would want to retain that).\n> \n> -Peff\n> \n\nThanks for advice. I will remember this. When I have time, I will look\nat git-send-mail eventually.\n\nPetr\n\n"}]}