public inbox for devel@edk2.groups.io
 help / color / mirror / Atom feed
From: "Laszlo Ersek" <lersek@redhat.com>
To: Andrew Fish <afish@apple.com>,
	devel@edk2.groups.io, Mike Kinney <michael.d.kinney@intel.com>,
	Leif Lindholm <leif.lindholm@linaro.org>
Cc: Jordan Justen <jordan.l.justen@intel.com>,
	"Feng, Bob C" <bob.c.feng@intel.com>,
	"Gao, Liming" <liming.gao@intel.com>
Subject: Re: [edk2-devel] [Patch 3/3] BaseTools/PatchCheck: Disable text conversion in 'git show'
Date: Fri, 9 Aug 2019 00:34:05 +0200	[thread overview]
Message-ID: <dae0b13e-db70-0929-3e55-e2a32800ba04@redhat.com> (raw)
In-Reply-To: <028D7F84-9970-450F-B6A9-4FEC8E5557D6@apple.com>

On 08/08/19 23:15, Andrew Fish wrote:
> 
> 
>> On Aug 8, 2019, at 2:08 PM, Michael D Kinney <michael.d.kinney@intel.com> wrote:
>>
>> Laszlo,
>>
>> This sounds like a good improvement.
>>
>> Do you have a recommended set of settings you would like
>> to see checked in to the root of each EDK II related 
>> repository that would be required settings for all EDK II
>> developers.
>>
> 
> Crazy idea allert ..... 
> 
> Could edksetup.sh have a --git option to set preferred git defaults?

Again, here I have to apologize to Leif: Leif has already contributed
the SetupGit.py script, which needs to be run only once, and then the
right settings are put in place permanently.

See the following two commits:

- 5b3e695d8ac5 ("BaseTools: add centralized location for git config
files", 2019-06-12)

- 4eb0acb1e2be ("BaseTools: add script to configure local git options",
2019-06-12)

The first commit adds:

  BaseTools/Conf/diff.order
  BaseTools/Conf/gitattributes

and the second commit adds:

  BaseTools/Scripts/SetupGit.py

Running the "SetupGit.py" script points configuration knobs permanently
to the files added in the preceding commit, plus puts other settings in
place (also permanently).

Again, my (somewhat) lame excuse for forgetting about these commits is
that I've never been *forced* to run the script myself. :)

Thanks
Laszlo

>> Do you want to enter a BZ and prepare a patch with this
>> new file for the edk2 repo.  Once it is approved and added
>> to edk2 repo, we can propagate to the other EDK II related
>> repos.
>>
>> Thanks,
>>
>> Mike
>>
>>> -----Original Message-----
>>> From: devel@edk2.groups.io <mailto:devel@edk2.groups.io>
>>> [mailto:devel@edk2.groups.io <mailto:devel@edk2.groups.io>] On Behalf Of Laszlo Ersek
>>> Sent: Thursday, August 8, 2019 1:58 PM
>>> To: devel@edk2.groups.io <mailto:devel@edk2.groups.io>; Justen, Jordan L
>>> <jordan.l.justen@intel.com <mailto:jordan.l.justen@intel.com>>; Kinney, Michael D
>>> <michael.d.kinney@intel.com <mailto:michael.d.kinney@intel.com>>
>>> Cc: Feng, Bob C <bob.c.feng@intel.com <mailto:bob.c.feng@intel.com>>; Gao, Liming
>>> <liming.gao@intel.com <mailto:liming.gao@intel.com>>
>>> Subject: Re: [edk2-devel] [Patch 3/3]
>>> BaseTools/PatchCheck: Disable text conversion in 'git
>>> show'
>>>
>>> On 08/07/19 21:42, Jordan Justen wrote:
>>>
>>>> I thought adding Laszlo's settings
>>>> to .gitattributes might amount to solving two issues
>>> with one change.
>>>
>>> Independently of the present patch, I would be very
>>> much in favor of tracking a .gitattributes file in the
>>> project root dir, *if* we could also automate the
>>> following setting:
>>>
>>> git config diff.ini.xfuncname     '^\[[A-Za-z0-9_.,
>>> ]+]'
>>>
>>> For me as a reviewer, that setting (on the submitter
>>> side) makes a huge difference. I keep asking people to
>>> do it manually, and that gets old really quick.
>>> Unfortunately, I don't know how to automate that config
>>> knob from within the source tree. :(
>>>
>>> Thanks,
>>> Laszlo
>>>
>>>
>>
>>
>> 
> 
> 


  reply	other threads:[~2019-08-08 22:34 UTC|newest]

Thread overview: 17+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2019-08-02  0:13 [Patch 0/3] BaseTools/PatchCheck: Fix false positives Michael D Kinney
2019-08-02  0:13 ` [Patch 1/3] BaseTools/PatchCheck: Ignore blank lines in diff Michael D Kinney
2019-08-02  0:13 ` [Patch 2/3] BaseTools/PatchCheck: Add copy from/to keywords Michael D Kinney
2019-08-02  0:13 ` [Patch 3/3] BaseTools/PatchCheck: Disable text conversion in 'git show' Michael D Kinney
2019-08-02 18:29   ` Jordan Justen
2019-08-02 18:51     ` Michael D Kinney
2019-08-02 23:57     ` Laszlo Ersek
2019-08-05 22:01       ` [edk2-devel] " Michael D Kinney
2019-08-07 13:26         ` Laszlo Ersek
2019-08-07 19:42         ` Jordan Justen
2019-08-08 20:57           ` Laszlo Ersek
2019-08-08 21:08             ` Michael D Kinney
2019-08-08 21:15               ` Andrew Fish
2019-08-08 22:34                 ` Laszlo Ersek [this message]
2019-08-02  7:59 ` [edk2-devel] [Patch 0/3] BaseTools/PatchCheck: Fix false positives Bob Feng
2019-08-02 18:30 ` Jordan Justen
2019-08-07 11:41 ` Liming Gao

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-list 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=dae0b13e-db70-0929-3e55-e2a32800ba04@redhat.com \
    --to=devel@edk2.groups.io \
    /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