From: "Miriam R." <mirucam@gmail.com>
To: Christian Couder <christian.couder@gmail.com>
Cc: git <git@vger.kernel.org>
Subject: Re: [Outreachy][PATCH] abspath: reconcile `dir_exists()` and `is_directory()`
Date: Sat, 26 Oct 2019 17:30:51 +0200 [thread overview]
Message-ID: <CAN7CjDDr0vDBDi+RKA0BMTHSaVQofc=GTCEuy1mAOaQmVqhJXA@mail.gmail.com> (raw)
In-Reply-To: <CAN7CjDC169rv8p9ZJcoLMeisXh7eMVcE4_-bpz8XFiYUsWAakQ@mail.gmail.com>
Dear all,
there is already a static function called path_exists() in archive.c
so project does not compile.
Maybe we could change the name of this static function and its
reference in archive.c like archive_path_exists() for example, or some
other you find more suitable.
Best,
Miriam
El vie., 25 oct. 2019 a las 17:23, Miriam R. (<mirucam@gmail.com>) escribió:
>
> Ok! Thanks to everyone.
>
> Best,
> Miriam
>
> El vie., 25 oct. 2019 a las 16:48, Christian Couder
> (<christian.couder@gmail.com>) escribió:
> >
> > On Fri, Oct 25, 2019 at 11:43 AM Junio C Hamano <gitster@pobox.com> wrote:
> > >
> > > "Miriam R." <mirucam@gmail.com> writes:
> > >
> > > > Ok, then after discussion, finally the issue tasks would be:
> > > >
> > > > - Add path_exists() that will work same as file_exists(), keeping for
> > > > now the latter.
> > > > - Use path_exists() instead of dir_exists() in builtin/clone.c.
> > >
> > > Sounds about right.
> > >
> > > > And also:
> > > > - Rename is_directory() to dir_exists(), as it is the equivalent to
> > > > path_exists()/file_exists(), isn't it?
> > >
> > > I wouldn't go there in the same series, if I were doing it. I'd
> > > expect that such a patch would be more noisy than it is worth if
> > > done in a single step. In order to avoid becoming a hindrance to
> > > other topics in flight, an ideal series to do so would support the
> > > same functionality with both old and new names, convert code that
> > > use the old name to use the new name, possibly in multiple patches
> > > to avoid unnecessary textual conflicts (i.e. some of these patches
> > > made to areas that are seeing active development will be discarded
> > > and need to be retried later when the area is more quiet) and then
> > > finally the function wither the old name gets removed.
> > >
> > > You would not want to mix the first two bullet points that are
> > > relatively isolated with such a long transition.
> >
> > Yeah, and for a micro-project it is more than enough if you only work
> > on the first two bullet points.
next prev parent reply other threads:[~2019-10-26 15:31 UTC|newest]
Thread overview: 15+ messages / expand[flat|nested] mbox.gz Atom feed top
2019-10-24 9:27 [Outreachy][PATCH] abspath: reconcile `dir_exists()` and `is_directory()` Miriam Rubio
2019-10-24 11:41 ` SZEDER Gábor
2019-10-24 18:13 ` Jeff King
2019-10-24 20:45 ` Emily Shaffer
2019-10-24 20:51 ` Jeff King
2019-10-25 2:40 ` Junio C Hamano
2019-10-24 20:57 ` Miriam R.
2019-10-25 2:45 ` Junio C Hamano
2019-10-25 8:59 ` Miriam R.
2019-10-25 9:43 ` Junio C Hamano
2019-10-25 14:47 ` Christian Couder
2019-10-25 15:23 ` Miriam R.
2019-10-26 15:30 ` Miriam R. [this message]
2019-10-26 18:05 ` Christian Couder
2019-10-26 18:42 ` Miriam R.
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to='CAN7CjDDr0vDBDi+RKA0BMTHSaVQofc=GTCEuy1mAOaQmVqhJXA@mail.gmail.com' \
--to=mirucam@gmail.com \
--cc=christian.couder@gmail.com \
--cc=git@vger.kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).