* [Buildroot] [PATCH] Fix swupdate build by installing mtd headers as expected @ 2020-07-10 19:41 Thomas W. 2020-07-10 19:55 ` Thomas W. 2020-07-11 11:05 ` Thomas Petazzoni 0 siblings, 2 replies; 5+ messages in thread From: Thomas W. @ 2020-07-10 19:41 UTC (permalink / raw) To: buildroot swupdate expects libmtd.h and libubi.h to be in the include folder and not in the mtd namespace. (see also: https://groups.google.com/forum/#!topic/swupdate/Apv_yRLmgzY https://groups.google.com/forum/#!topic/swupdate/gd25RIPa00w https://groups.google.com/forum/#!topic/swupdate/gei0Gc_5j8M) Signed-off-by: Thomas Chiantia <thomas.chiantia@gmail.com> --- ^ permalink raw reply [flat|nested] 5+ messages in thread
* [Buildroot] [PATCH] Fix swupdate build by installing mtd headers as expected 2020-07-10 19:41 [Buildroot] [PATCH] Fix swupdate build by installing mtd headers as expected Thomas W. @ 2020-07-10 19:55 ` Thomas W. 2020-07-11 11:05 ` Thomas Petazzoni 1 sibling, 0 replies; 5+ messages in thread From: Thomas W. @ 2020-07-10 19:55 UTC (permalink / raw) To: buildroot (Also bump swupdate package) Signed-off-by: Thomas Chiantia <thomas.chiantia@gmail.com> --- ^ permalink raw reply [flat|nested] 5+ messages in thread
* [Buildroot] [PATCH] Fix swupdate build by installing mtd headers as expected 2020-07-10 19:41 [Buildroot] [PATCH] Fix swupdate build by installing mtd headers as expected Thomas W. 2020-07-10 19:55 ` Thomas W. @ 2020-07-11 11:05 ` Thomas Petazzoni 2020-07-11 17:26 ` Thomas W. 1 sibling, 1 reply; 5+ messages in thread From: Thomas Petazzoni @ 2020-07-11 11:05 UTC (permalink / raw) To: buildroot Hello Thomas, Thanks for your contribution! On Fri, 10 Jul 2020 15:41:37 -0400 "Thomas W." <thomas.chiantia@gmail.com> wrote: > swupdate expects libmtd.h and libubi.h to be in the include folder and > not in the mtd namespace. > (see also: > https://groups.google.com/forum/#!topic/swupdate/Apv_yRLmgzY > https://groups.google.com/forum/#!topic/swupdate/gd25RIPa00w > https://groups.google.com/forum/#!topic/swupdate/gei0Gc_5j8M) > > Signed-off-by: Thomas Chiantia <thomas.chiantia@gmail.com> Which problem is this fixing ? Is there a build issue today ? We have had mtd + swupdate for quite some time. Why suddenly should we change this ? > diff --git a/package/mtd/mtd.mk b/package/mtd/mtd.mk > index 3477460200..76ede98879 100644 > --- a/package/mtd/mtd.mk > +++ b/package/mtd/mtd.mk > @@ -119,8 +119,8 @@ endef > # Those libraries are not installed by "make install", but are needed > # by other packages, such as swupdate. > define MTD_INSTALL_LIBS > - $(INSTALL) -D -m 0755 $(@D)/include/libmtd.h > $(STAGING_DIR)/usr/include/mtd/libmtd.h > - $(INSTALL) -D -m 0755 $(@D)/include/libubi.h > $(STAGING_DIR)/usr/include/mtd/libubi.h > + $(INSTALL) -D -m 0755 $(@D)/include/libmtd.h > $(STAGING_DIR)/usr/include/libmtd.h > + $(INSTALL) -D -m 0755 $(@D)/include/libubi.h > $(STAGING_DIR)/usr/include/libubi.h > $(INSTALL) -D -m 0755 $(@D)/include/mtd/ubi-media.h > $(STAGING_DIR)/usr/include/mtd/ubi-media.h > $(INSTALL) -D -m 0755 $(@D)/libmtd.a $(STAGING_DIR)/usr/lib/libmtd.a > $(INSTALL) -D -m 0755 $(@D)/libubi.a $(STAGING_DIR)/usr/lib/libubi.a Your patch is badly line-wrapped by your e-mail client. Could you send your patches using git send-email ? Thanks! Thomas -- Thomas Petazzoni, CTO, Bootlin Embedded Linux and Kernel engineering https://bootlin.com ^ permalink raw reply [flat|nested] 5+ messages in thread
* [Buildroot] [PATCH] Fix swupdate build by installing mtd headers as expected 2020-07-11 11:05 ` Thomas Petazzoni @ 2020-07-11 17:26 ` Thomas W. 2020-07-11 17:32 ` Thomas W. 0 siblings, 1 reply; 5+ messages in thread From: Thomas W. @ 2020-07-11 17:26 UTC (permalink / raw) To: buildroot > Which problem is this fixing ? Is there a build issue today ? We have > had mtd + swupdate for quite some time. Why suddenly should we change > this ? Hello, There is no issue with the current Buildroot. Swupdate-2019.11 expects MTD headers in /include/mtd folder, and swupdate-2020.4 expects MTD headers in /include folder. I upgraded my swupdate to 2020.4 and found that the MTD header installation must change. I tried sending PR to swupdate to include MTD headers in the original /include/mtd folder but they said it should change in buildroot. I don't know if other projects depend on the mtd headers. If other projects depend on the MTD headers then they might break with this patch unless they are aware of this change. On Sat, Jul 11, 2020 at 7:05 AM Thomas Petazzoni <thomas.petazzoni@bootlin.com> wrote: > > Hello Thomas, > > Thanks for your contribution! > > On Fri, 10 Jul 2020 15:41:37 -0400 > "Thomas W." <thomas.chiantia@gmail.com> wrote: > > > swupdate expects libmtd.h and libubi.h to be in the include folder and > > not in the mtd namespace. > > (see also: > > https://groups.google.com/forum/#!topic/swupdate/Apv_yRLmgzY > > https://groups.google.com/forum/#!topic/swupdate/gd25RIPa00w > > https://groups.google.com/forum/#!topic/swupdate/gei0Gc_5j8M) > > > > Signed-off-by: Thomas Chiantia <thomas.chiantia@gmail.com> > > Which problem is this fixing ? Is there a build issue today ? We have > had mtd + swupdate for quite some time. Why suddenly should we change > this ? > > > diff --git a/package/mtd/mtd.mk b/package/mtd/mtd.mk > > index 3477460200..76ede98879 100644 > > --- a/package/mtd/mtd.mk > > +++ b/package/mtd/mtd.mk > > @@ -119,8 +119,8 @@ endef > > # Those libraries are not installed by "make install", but are needed > > # by other packages, such as swupdate. > > define MTD_INSTALL_LIBS > > - $(INSTALL) -D -m 0755 $(@D)/include/libmtd.h > > $(STAGING_DIR)/usr/include/mtd/libmtd.h > > - $(INSTALL) -D -m 0755 $(@D)/include/libubi.h > > $(STAGING_DIR)/usr/include/mtd/libubi.h > > + $(INSTALL) -D -m 0755 $(@D)/include/libmtd.h > > $(STAGING_DIR)/usr/include/libmtd.h > > + $(INSTALL) -D -m 0755 $(@D)/include/libubi.h > > $(STAGING_DIR)/usr/include/libubi.h > > $(INSTALL) -D -m 0755 $(@D)/include/mtd/ubi-media.h > > $(STAGING_DIR)/usr/include/mtd/ubi-media.h > > $(INSTALL) -D -m 0755 $(@D)/libmtd.a $(STAGING_DIR)/usr/lib/libmtd.a > > $(INSTALL) -D -m 0755 $(@D)/libubi.a $(STAGING_DIR)/usr/lib/libubi.a > > Your patch is badly line-wrapped by your e-mail client. Could you send > your patches using git send-email ? > > Thanks! > > Thomas > -- > Thomas Petazzoni, CTO, Bootlin > Embedded Linux and Kernel engineering > https://bootlin.com ^ permalink raw reply [flat|nested] 5+ messages in thread
* [Buildroot] [PATCH] Fix swupdate build by installing mtd headers as expected 2020-07-11 17:26 ` Thomas W. @ 2020-07-11 17:32 ` Thomas W. 0 siblings, 0 replies; 5+ messages in thread From: Thomas W. @ 2020-07-11 17:32 UTC (permalink / raw) To: buildroot > Your patch is badly line-wrapped by your e-mail client. Could you send > your patches using git send-email ? Sorry, I will try to send it again if you want to update swupdate (and it is found that other packages will not be broken with this patch). On Sat, Jul 11, 2020 at 1:26 PM Thomas W. <thomas.chiantia@gmail.com> wrote: > > > Which problem is this fixing ? Is there a build issue today ? We have > > had mtd + swupdate for quite some time. Why suddenly should we change > > this ? > > Hello, > > There is no issue with the current Buildroot. Swupdate-2019.11 expects > MTD headers in /include/mtd folder, and swupdate-2020.4 expects MTD > headers in /include folder. I upgraded my swupdate to 2020.4 and > found that the MTD header installation must change. I tried sending PR > to swupdate to include MTD headers in the original /include/mtd folder > but they said it should change in buildroot. I don't know if other > projects depend on the mtd headers. If other projects depend on the > MTD headers then they might break with this patch unless they are > aware of this change. > > On Sat, Jul 11, 2020 at 7:05 AM Thomas Petazzoni > <thomas.petazzoni@bootlin.com> wrote: > > > > Hello Thomas, > > > > Thanks for your contribution! > > > > On Fri, 10 Jul 2020 15:41:37 -0400 > > "Thomas W." <thomas.chiantia@gmail.com> wrote: > > > > > swupdate expects libmtd.h and libubi.h to be in the include folder and > > > not in the mtd namespace. > > > (see also: > > > https://groups.google.com/forum/#!topic/swupdate/Apv_yRLmgzY > > > https://groups.google.com/forum/#!topic/swupdate/gd25RIPa00w > > > https://groups.google.com/forum/#!topic/swupdate/gei0Gc_5j8M) > > > > > > Signed-off-by: Thomas Chiantia <thomas.chiantia@gmail.com> > > > > Which problem is this fixing ? Is there a build issue today ? We have > > had mtd + swupdate for quite some time. Why suddenly should we change > > this ? > > > > > diff --git a/package/mtd/mtd.mk b/package/mtd/mtd.mk > > > index 3477460200..76ede98879 100644 > > > --- a/package/mtd/mtd.mk > > > +++ b/package/mtd/mtd.mk > > > @@ -119,8 +119,8 @@ endef > > > # Those libraries are not installed by "make install", but are needed > > > # by other packages, such as swupdate. > > > define MTD_INSTALL_LIBS > > > - $(INSTALL) -D -m 0755 $(@D)/include/libmtd.h > > > $(STAGING_DIR)/usr/include/mtd/libmtd.h > > > - $(INSTALL) -D -m 0755 $(@D)/include/libubi.h > > > $(STAGING_DIR)/usr/include/mtd/libubi.h > > > + $(INSTALL) -D -m 0755 $(@D)/include/libmtd.h > > > $(STAGING_DIR)/usr/include/libmtd.h > > > + $(INSTALL) -D -m 0755 $(@D)/include/libubi.h > > > $(STAGING_DIR)/usr/include/libubi.h > > > $(INSTALL) -D -m 0755 $(@D)/include/mtd/ubi-media.h > > > $(STAGING_DIR)/usr/include/mtd/ubi-media.h > > > $(INSTALL) -D -m 0755 $(@D)/libmtd.a $(STAGING_DIR)/usr/lib/libmtd.a > > > $(INSTALL) -D -m 0755 $(@D)/libubi.a $(STAGING_DIR)/usr/lib/libubi.a > > > > Your patch is badly line-wrapped by your e-mail client. Could you send > > your patches using git send-email ? > > > > Thanks! > > > > Thomas > > -- > > Thomas Petazzoni, CTO, Bootlin > > Embedded Linux and Kernel engineering > > https://bootlin.com ^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2020-07-11 17:32 UTC | newest] Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2020-07-10 19:41 [Buildroot] [PATCH] Fix swupdate build by installing mtd headers as expected Thomas W. 2020-07-10 19:55 ` Thomas W. 2020-07-11 11:05 ` Thomas Petazzoni 2020-07-11 17:26 ` Thomas W. 2020-07-11 17:32 ` Thomas W.
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.