| Message ID | 1405655815-2025-2-git-send-email-danomimanchego123@gmail.com |
|---|---|
| State | Superseded |
| Headers | show |
Dear Danomi Manchego, On Thu, 17 Jul 2014 23:56:54 -0400, Danomi Manchego wrote: > During the first compile, the keyword-gen check is not run. However, > it gets run on the first rebuild. Because we are cross-compiling, the > keyword-gen can't run on the host. Since this is only used to check if > ntp_keyword.h is up to date, we simply drop the check altogether. > > Signed-off-by: Danomi Manchego <danomimanchego123@gmail.com> I haven't applied this one, because I'm not entirely satisfied by the explanation here. Why isn't the keyword-gen check executed on the first run? I did some testing, and I indeed don't see the rule about ntp_keyword.h being evaluated on the first build. Do you have more details about what makes the difference between the first and subsequent builds? Thanks, Thomas
On Fri, Jul 18, 2014 at 3:26 PM, Thomas Petazzoni <thomas.petazzoni@free-electrons.com> wrote: > Dear Danomi Manchego, > > On Thu, 17 Jul 2014 23:56:54 -0400, Danomi Manchego wrote: >> During the first compile, the keyword-gen check is not run. However, >> it gets run on the first rebuild. Because we are cross-compiling, the >> keyword-gen can't run on the host. Since this is only used to check if >> ntp_keyword.h is up to date, we simply drop the check altogether. >> >> Signed-off-by: Danomi Manchego <danomimanchego123@gmail.com> > > I haven't applied this one, because I'm not entirely satisfied by the > explanation here. Why isn't the keyword-gen check executed on the first > run? I did some testing, and I indeed don't see the rule about > ntp_keyword.h being evaluated on the first build. Do you have more > details about what makes the difference between the first and > subsequent builds? No. Stranger yet, when the patch is applied, I found that the keyword-gen check is only run on the *first* ntp-rebuild. Further ntp-rebuilds do not cause the check to re-occur. But anyway, I didn't notice any indication that the keyword-gen would be compiled with a HOSTCC, so I reasoned that if it's cross-compiled, then it shouldn't be run on the build machine in any circumstance. So I didn't investigate further. Danomi -
Dear Danomi Manchego, On Fri, 18 Jul 2014 20:52:18 -0400, Danomi Manchego wrote: > > I haven't applied this one, because I'm not entirely satisfied by the > > explanation here. Why isn't the keyword-gen check executed on the first > > run? I did some testing, and I indeed don't see the rule about > > ntp_keyword.h being evaluated on the first build. Do you have more > > details about what makes the difference between the first and > > subsequent builds? > > No. Stranger yet, when the patch is applied, I found that the > keyword-gen check is only run on the *first* ntp-rebuild. Further > ntp-rebuilds do not cause the check to re-occur. > > But anyway, I didn't notice any indication that the keyword-gen would > be compiled with a HOSTCC, so I reasoned that if it's cross-compiled, > then it shouldn't be run on the build machine in any circumstance. So > I didn't investigate further. Right, but the ntp source code comes with a pre-generated result of this keyword-gen program, which allows the first build to work properly. So it'd be good to understand why this logic used for the first build doesn't work for the subsequent builds. Thanks, Thomas
diff --git a/package/ntp/ntp-003-skip-keyword-check.patch b/package/ntp/ntp-003-skip-keyword-check.patch new file mode 100644 index 0000000..dc09fbc --- /dev/null +++ b/package/ntp/ntp-003-skip-keyword-check.patch @@ -0,0 +1,56 @@ +ntp: prevent keyword-gen failure during ntp_keyword.h check + +During the first compile, the keyword-gen check is not run. However, +it gets run on the first rebuild. Because we are cross-compiling, the +keyword-gen can't run on the host. Since this is only used to check if +ntp_keyword.h is up to date, we simply drop the check altogether. + +Signed-off-by: Danomi Manchego <danomimanchego123@gmail.com> + + +diff -purN a/ntpd/Makefile.am b/ntpd/Makefile.am +--- a/ntpd/Makefile.am 2014-07-16 15:16:30.650280205 -0400 ++++ b/ntpd/Makefile.am 2014-07-16 15:24:57.256527446 -0400 +@@ -213,17 +213,8 @@ libntpd_a_SOURCES = \ + refclock_zyfer.c \ + $(NULL) + +-k-g-u-submake: keyword-gen +- ./keyword-gen $(srcdir)/ntp_parser.h > k-g.out +- @grep -v diff_ignore_line < k-g.out > cmp1 +- @grep -v diff_ignore_line < $(srcdir)/ntp_keyword.h > cmp2 +- @cmp cmp1 cmp2 > /dev/null || \ +- { mv -f k-g.out $(srcdir)/ntp_keyword.h && \ +- echo 'Generated changed ntp_keyword.h.' ;} +- @[ ! -f k-g.out ] || \ +- { rm k-g.out && echo 'ntp_keyword.h is up to date.' ;} +- @rm cmp1 cmp2 +- @echo 'keyword-gen and ntp_keyword.h are up to date.' > $@ ++k-g-u-submake: ++ # Skip keyword-gen check due to cross-compile + + $(srcdir)/keyword-gen-utd: $(srcdir)/keyword-gen.c $(srcdir)/ntp_parser.h + $(MAKE) k-g-u-submake # avoid explicit dependency +diff -purN a/ntpd/Makefile.in b/ntpd/Makefile.in +--- a/ntpd/Makefile.in 2014-07-16 15:16:30.650280205 -0400 ++++ b/ntpd/Makefile.in 2014-07-16 15:25:08.764962071 -0400 +@@ -1182,17 +1182,8 @@ vphack_after: + check-local: @MAKE_CHECK_Y2K@ $(CHECK_SAVECONFIG) + test -z "@MAKE_CHECK_Y2K@" || ./@MAKE_CHECK_Y2K@ + +-k-g-u-submake: keyword-gen +- ./keyword-gen $(srcdir)/ntp_parser.h > k-g.out +- @grep -v diff_ignore_line < k-g.out > cmp1 +- @grep -v diff_ignore_line < $(srcdir)/ntp_keyword.h > cmp2 +- @cmp cmp1 cmp2 > /dev/null || \ +- { mv -f k-g.out $(srcdir)/ntp_keyword.h && \ +- echo 'Generated changed ntp_keyword.h.' ;} +- @[ ! -f k-g.out ] || \ +- { rm k-g.out && echo 'ntp_keyword.h is up to date.' ;} +- @rm cmp1 cmp2 +- @echo 'keyword-gen and ntp_keyword.h are up to date.' > $@ ++k-g-u-submake: ++ # Skip keyword-gen check due to cross-compile + + $(srcdir)/keyword-gen-utd: $(srcdir)/keyword-gen.c $(srcdir)/ntp_parser.h + $(MAKE) k-g-u-submake # avoid explicit dependency
During the first compile, the keyword-gen check is not run. However, it gets run on the first rebuild. Because we are cross-compiling, the keyword-gen can't run on the host. Since this is only used to check if ntp_keyword.h is up to date, we simply drop the check altogether. Signed-off-by: Danomi Manchego <danomimanchego123@gmail.com> --- Note: "the first rebuild" means "the first 'make ntp-rebuild'" --- package/ntp/ntp-003-skip-keyword-check.patch | 56 ++++++++++++++++++++++++++ 1 file changed, 56 insertions(+) create mode 100644 package/ntp/ntp-003-skip-keyword-check.patch