From: Masahiro Yamada <yamada.masahiro@socionext.com> To: "Theodore Y. Ts'o" <tytso@mit.edu>, "Enrico Weigelt, metux IT consult" <lkml@metux.net>, Masahiro Yamada <yamada.masahiro@socionext.com>, "Enrico Weigelt, metux IT consult" <info@metux.net>, Linux Kernel Mailing List <linux-kernel@vger.kernel.org>, Michal Marek <michal.lkml@markovi.net>, Robo Bot <apw@canonical.com>, Joe Perches <joe@perches.com>, Linux Kbuild mailing list <linux-kbuild@vger.kernel.org>, linux-riscv@lists.infradead.org, clang-built-linux <clang-built-linux@googlegroups.com> Subject: Re: [PATCH 4/4] debian: add generic rule file Date: Tue, 16 Jul 2019 17:58:49 +0900 [thread overview] Message-ID: <CAK7LNASps6JBAvtJshjMbqMk8QaSrMaH8pm-wHsEySTRJzu0Kw@mail.gmail.com> (raw) In-Reply-To: <20190715191245.GD3068@mit.edu> On Tue, Jul 16, 2019 at 4:13 AM Theodore Y. Ts'o <tytso@mit.edu> wrote: > > On Mon, Jul 15, 2019 at 08:56:25PM +0200, Enrico Weigelt, metux IT consult wrote: > > On 15.07.19 14:28, Masahiro Yamada wrote: > > > > >> The rule file contains a rule for creating debian/control and > > >> other metadata - this is done similar to the 'deb-pkg' make rule, > > >> scripts/packaging/mkdebian. > > > > > > I saw a similar patch submission before, and negative feedback about it. > > > > Do you recall what negative feedback exactly ? Sorry, my memory was broken. I did not like this patch set from the beginning, but missed to express my opinion strongly. I want debian/ to be kept as a drop-in directory for packagers, without replacing the upstream debian/rules. If a check-in source file is modified in anyway, scripts/setlocalversion would set -dirty flag, which I want to avoid. > It's possible I'm not remembering some of the feedback, but the only > thing I recall was the comment I made that I'd really like this use > case: > > make O=/build/linux-build bindeb-pkg > > to not break. And as far as I can tell from the proposed patch series > (I haven't had a chance to experimentally verify it yet), I don't > think it should break anything --- I'm assuming that we will still > have a way of creating the debian/rules file in > /build/linux-build/debian/rules when doing a O= build, and that the > intdeb-pkg rule remains the same. At least, it appears to be the case > from my doing a quick look at the patches. > > > > Debian maintains its own debian/rules, and it is fine. > > > > Not for me, I don't use it - given up trying to make anything useful > > out of it. It's extremly complex, practically undebuggable and doesn't > > even work w/o lots of external preparations. > > Yeah, the official Debian debian/rules is optimized for doing a > distribution release, and in addition to the issues Enrico has raised, > last time I tried it, it was S-L-O-W since it was building a fully > generic kernel. It's not at all useable for general developer use. It is OK if the package is targeting normal users instead of kernel developers. > It sounds like what Enrico is trying to do is to enable running > "dpkg-buildpackage -us -uc -b" from the the top-level kernel package > as being easier than running "make bindeb-pkg". I suspect this might > be because his goal is to integrate individual kernel builds from > using Debian's hermetic build / chroot systems (e.g., sbuild, pbuilder)? I am OK with generating debian/rules with 'make bindeb-pkg', a shell scripts or whatever, but I dislike to commit it in upstream git tree. debian/rules is a hook for packagers to do their jobs in downstream. "We kindly committed a generic one for you" sounds weird to me. -- Best Regards Masahiro Yamada
WARNING: multiple messages have this Message-ID (diff)
From: Masahiro Yamada <yamada.masahiro@socionext.com> To: "Theodore Y. Ts'o" <tytso@mit.edu>, "Enrico Weigelt, metux IT consult" <lkml@metux.net>, Masahiro Yamada <yamada.masahiro@socionext.com>, "Enrico Weigelt, metux IT consult" <info@metux.net>, Linux Kernel Mailing List <linux-kernel@vger.kernel.org>, Michal Marek <michal.lkml@markovi.net>, Robo Bot <apw@canonical.com>, Joe Perches <joe@perches.com>, Linux Kbuild mailing list <linux-kbuild@vger.kernel.org>, linux-riscv@lists.infradead.org, clang-built-linux <clang-built-linux@googlegroups.com> Subject: Re: [PATCH 4/4] debian: add generic rule file Date: Tue, 16 Jul 2019 17:58:49 +0900 [thread overview] Message-ID: <CAK7LNASps6JBAvtJshjMbqMk8QaSrMaH8pm-wHsEySTRJzu0Kw@mail.gmail.com> (raw) In-Reply-To: <20190715191245.GD3068@mit.edu> On Tue, Jul 16, 2019 at 4:13 AM Theodore Y. Ts'o <tytso@mit.edu> wrote: > > On Mon, Jul 15, 2019 at 08:56:25PM +0200, Enrico Weigelt, metux IT consult wrote: > > On 15.07.19 14:28, Masahiro Yamada wrote: > > > > >> The rule file contains a rule for creating debian/control and > > >> other metadata - this is done similar to the 'deb-pkg' make rule, > > >> scripts/packaging/mkdebian. > > > > > > I saw a similar patch submission before, and negative feedback about it. > > > > Do you recall what negative feedback exactly ? Sorry, my memory was broken. I did not like this patch set from the beginning, but missed to express my opinion strongly. I want debian/ to be kept as a drop-in directory for packagers, without replacing the upstream debian/rules. If a check-in source file is modified in anyway, scripts/setlocalversion would set -dirty flag, which I want to avoid. > It's possible I'm not remembering some of the feedback, but the only > thing I recall was the comment I made that I'd really like this use > case: > > make O=/build/linux-build bindeb-pkg > > to not break. And as far as I can tell from the proposed patch series > (I haven't had a chance to experimentally verify it yet), I don't > think it should break anything --- I'm assuming that we will still > have a way of creating the debian/rules file in > /build/linux-build/debian/rules when doing a O= build, and that the > intdeb-pkg rule remains the same. At least, it appears to be the case > from my doing a quick look at the patches. > > > > Debian maintains its own debian/rules, and it is fine. > > > > Not for me, I don't use it - given up trying to make anything useful > > out of it. It's extremly complex, practically undebuggable and doesn't > > even work w/o lots of external preparations. > > Yeah, the official Debian debian/rules is optimized for doing a > distribution release, and in addition to the issues Enrico has raised, > last time I tried it, it was S-L-O-W since it was building a fully > generic kernel. It's not at all useable for general developer use. It is OK if the package is targeting normal users instead of kernel developers. > It sounds like what Enrico is trying to do is to enable running > "dpkg-buildpackage -us -uc -b" from the the top-level kernel package > as being easier than running "make bindeb-pkg". I suspect this might > be because his goal is to integrate individual kernel builds from > using Debian's hermetic build / chroot systems (e.g., sbuild, pbuilder)? I am OK with generating debian/rules with 'make bindeb-pkg', a shell scripts or whatever, but I dislike to commit it in upstream git tree. debian/rules is a hook for packagers to do their jobs in downstream. "We kindly committed a generic one for you" sounds weird to me. -- Best Regards Masahiro Yamada _______________________________________________ linux-riscv mailing list linux-riscv@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-riscv
next prev parent reply other threads:[~2019-07-16 8:59 UTC|newest] Thread overview: 30+ messages / expand[flat|nested] mbox.gz Atom feed top 2019-07-09 9:32 [PATCH 1/4] Makefile: rules for printing kernel architecture and localversion Enrico Weigelt, metux IT consult 2019-07-09 9:32 ` Enrico Weigelt, metux IT consult 2019-07-09 9:32 ` [PATCH 2/4] scripts: mkdebian: allow renaming generated debian/rules via env Enrico Weigelt, metux IT consult 2019-07-09 9:32 ` Enrico Weigelt, metux IT consult 2019-07-09 9:32 ` [PATCH 3/4] scripts: checkpatch.pl: don't complain that debian/rules is executable Enrico Weigelt, metux IT consult 2019-07-09 9:32 ` Enrico Weigelt, metux IT consult 2019-07-09 10:16 ` Joe Perches 2019-07-09 10:16 ` Joe Perches 2019-07-09 9:32 ` [PATCH 4/4] debian: add generic rule file Enrico Weigelt, metux IT consult 2019-07-09 9:32 ` Enrico Weigelt, metux IT consult 2019-07-15 12:28 ` Masahiro Yamada 2019-07-15 12:28 ` Masahiro Yamada 2019-07-15 18:56 ` Enrico Weigelt, metux IT consult 2019-07-15 18:56 ` Enrico Weigelt, metux IT consult 2019-07-15 19:12 ` Theodore Y. Ts'o 2019-07-15 19:12 ` Theodore Y. Ts'o 2019-07-16 8:58 ` Masahiro Yamada [this message] 2019-07-16 8:58 ` Masahiro Yamada 2019-07-16 12:34 ` Theodore Y. Ts'o 2019-07-16 12:34 ` Theodore Y. Ts'o 2019-07-17 14:16 ` Enrico Weigelt, metux IT consult 2019-07-17 14:16 ` Enrico Weigelt, metux IT consult 2019-07-17 15:23 ` Theodore Y. Ts'o 2019-07-17 15:23 ` Theodore Y. Ts'o 2019-07-17 11:08 ` Enrico Weigelt, metux IT consult 2019-07-17 11:08 ` Enrico Weigelt, metux IT consult 2019-07-09 14:54 ` [PATCH 1/4] Makefile: rules for printing kernel architecture and localversion Randy Dunlap 2019-07-09 14:54 ` Randy Dunlap -- strict thread matches above, loose matches on Subject: below -- 2019-02-07 16:40 RFC: refactoring the deb build Enrico Weigelt, metux IT consult 2019-02-07 16:40 ` [PATCH 4/4] debian: add generic rule file Enrico Weigelt, metux IT consult 2019-02-07 16:40 ` Enrico Weigelt, metux IT consult
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=CAK7LNASps6JBAvtJshjMbqMk8QaSrMaH8pm-wHsEySTRJzu0Kw@mail.gmail.com \ --to=yamada.masahiro@socionext.com \ --cc=apw@canonical.com \ --cc=clang-built-linux@googlegroups.com \ --cc=info@metux.net \ --cc=joe@perches.com \ --cc=linux-kbuild@vger.kernel.org \ --cc=linux-kernel@vger.kernel.org \ --cc=linux-riscv@lists.infradead.org \ --cc=lkml@metux.net \ --cc=michal.lkml@markovi.net \ --cc=tytso@mit.edu \ /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: linkBe sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.