From: Daniel Thompson <daniel.thompson@linaro.org>
To: Jan Kiszka <jan.kiszka@siemens.com>,
Michal Marek <mmarek@suse.cz>,
Andrew Morton <akpm@linux-foundation.org>,
linux-kernel@vger.kernel.org
Cc: Thomas Gleixner <tglx@linutronix.de>,
Jason Wessel <jason.wessel@windriver.com>,
kgdb-bugreport@lists.sourceforge.net,
Andi Kleen <andi@firstfloor.org>, Tom Tromey <tromey@redhat.com>,
Ben Widawsky <ben@bwidawsk.net>, Borislav Petkov <bp@suse.de>,
linux-kbuild@vger.kernel.org
Subject: Re: [PATCH v11 23/28] scripts/gdb: Ignore byte-compiled python files
Date: Thu, 29 Jan 2015 14:34:10 +0000 [thread overview]
Message-ID: <54CA44E2.904@linaro.org> (raw)
In-Reply-To: <54CA3736.4090600@siemens.com>
On 29/01/15 13:35, Jan Kiszka wrote:
> On 2015-01-29 14:15, Michal Marek wrote:
>> On 2015-01-29 07:46, Jan Kiszka wrote:
>>> --- a/Makefile
>>> +++ b/Makefile
>>> @@ -1179,7 +1179,8 @@ MRPROPER_FILES += .config .config.old .version .old_version $(version_h) \
>>> Module.symvers tags TAGS cscope* GPATH GTAGS GRTAGS GSYMS \
>>> signing_key.priv signing_key.x509 x509.genkey \
>>> extra_certificates signing_key.x509.keyid \
>>> - signing_key.x509.signer vmlinux-gdb.py
>>> + signing_key.x509.signer vmlinux-gdb.py \
>>> + scripts/gdb/linux/*.py[co]
>>
>> In 01/28, you already have
>>
>> +clean-files := *.pyc $(if $(KBUILD_SRC),*.py)
>>
>> Maybe just add *.pyo there and you don't need to add it to the toplevel
>> Makefile.
>
> Based on a quick test, you seem right. Daniel, anything we miss?
No. Deleting it with the clean rule makes sense.
Of course the .gitignore parts of my patch will still be needed in some
form or other.
Daniel.
next prev parent reply other threads:[~2015-01-29 14:34 UTC|newest]
Thread overview: 9+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-01-29 6:46 [PATCH v11 00/28] Add gdb python scripts as kernel debugging helpers Jan Kiszka
2015-01-29 6:46 ` [PATCH v11 01/28] scripts/gdb: Add infrastructure Jan Kiszka
2015-01-29 13:23 ` Michal Marek
2015-01-29 13:37 ` Jan Kiszka
2015-01-29 6:46 ` [PATCH v11 23/28] scripts/gdb: Ignore byte-compiled python files Jan Kiszka
2015-01-29 13:15 ` Michal Marek
2015-01-29 13:35 ` Jan Kiszka
2015-01-29 14:34 ` Daniel Thompson [this message]
2015-01-29 14:41 ` Jan Kiszka
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=54CA44E2.904@linaro.org \
--to=daniel.thompson@linaro.org \
--cc=akpm@linux-foundation.org \
--cc=andi@firstfloor.org \
--cc=ben@bwidawsk.net \
--cc=bp@suse.de \
--cc=jan.kiszka@siemens.com \
--cc=jason.wessel@windriver.com \
--cc=kgdb-bugreport@lists.sourceforge.net \
--cc=linux-kbuild@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=mmarek@suse.cz \
--cc=tglx@linutronix.de \
--cc=tromey@redhat.com \
/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