Message ID | 1353273224-14685-1-git-send-email-zinosat@tiscali.it |
---|---|
State | Superseded |
Headers | show |
Davide, Samuel, All, On Sunday 18 November 2012 Davide Viti wrote: > The list was obtained with the following command: > grep "_HOOKS[ ]\+?=" package/pkg-generic.mk This patch is the patch in bug #5726: https://bugs.busybox.net/show_bug.cgi?id=5726 Samuel, this is a documentation fix for the manual: I think it should be applied for 2012.11. Acked-by: "Yann E. MORIN" <yann.morin.1998@free.fr> (Note: Samuel: this is your first act as doc maintainer! ;-) ) Regards, Yann E. MORIN.
Hi Davide, all, 2012/11/18 Davide Viti <zinosat@tiscali.it>: > The list was obtained with the following command: > grep "_HOOKS[ ]\+?=" package/pkg-generic.mk > --- > docs/manual/adding-packages-generic.txt | 4 ++++ > 1 files changed, 4 insertions(+), 0 deletions(-) > > diff --git a/docs/manual/adding-packages-generic.txt b/docs/manual/adding-packages-generic.txt > index b05043a..a2f37ec 100644 > --- a/docs/manual/adding-packages-generic.txt > +++ b/docs/manual/adding-packages-generic.txt > @@ -357,6 +357,9 @@ userful for generic packages. > > The following hook points are available: > > +* +LIBFOO_POST_DOWNLOAD_HOOKS+ > +* +LIBFOO_POST_EXTRACT_HOOKS+ > +* +LIBFOO_PRE_PATCH_HOOKS+ > * +LIBFOO_POST_PATCH_HOOKS+ > * +LIBFOO_PRE_CONFIGURE_HOOKS+ > * +LIBFOO_POST_CONFIGURE_HOOKS+ > @@ -364,6 +367,7 @@ The following hook points are available: > * +LIBFOO_POST_INSTALL_HOOKS+ (for host packages only) > * +LIBFOO_POST_INSTALL_STAGING_HOOKS+ (for target packages only) > * +LIBFOO_POST_INSTALL_TARGET_HOOKS+ (for target packages only) > +* +LIBFOO_POST_INSTALL_IMAGES_HOOKS+ I'm a bit worried about this POST_INSTALL_IMAGES_HOOKS could be mis-understood or mis-used... Maybe it'd be worth to add few words saying that it's mainly for bootloader purpose. After that, you can add my ack: Acked-by: Samuel Martin <s.martin49@gmail.com> Regards,
Hi Davide, Yann, all, 2012/11/18 Samuel Martin <s.martin49@gmail.com>: > Hi Davide, all, > > 2012/11/18 Davide Viti <zinosat@tiscali.it>: >> The list was obtained with the following command: >> grep "_HOOKS[ ]\+?=" package/pkg-generic.mk >> --- >> docs/manual/adding-packages-generic.txt | 4 ++++ >> 1 files changed, 4 insertions(+), 0 deletions(-) >> >> diff --git a/docs/manual/adding-packages-generic.txt b/docs/manual/adding-packages-generic.txt >> index b05043a..a2f37ec 100644 >> --- a/docs/manual/adding-packages-generic.txt >> +++ b/docs/manual/adding-packages-generic.txt >> @@ -357,6 +357,9 @@ userful for generic packages. >> >> The following hook points are available: >> >> +* +LIBFOO_POST_DOWNLOAD_HOOKS+ >> +* +LIBFOO_POST_EXTRACT_HOOKS+ >> +* +LIBFOO_PRE_PATCH_HOOKS+ >> * +LIBFOO_POST_PATCH_HOOKS+ >> * +LIBFOO_PRE_CONFIGURE_HOOKS+ >> * +LIBFOO_POST_CONFIGURE_HOOKS+ >> @@ -364,6 +367,7 @@ The following hook points are available: >> * +LIBFOO_POST_INSTALL_HOOKS+ (for host packages only) >> * +LIBFOO_POST_INSTALL_STAGING_HOOKS+ (for target packages only) >> * +LIBFOO_POST_INSTALL_TARGET_HOOKS+ (for target packages only) >> +* +LIBFOO_POST_INSTALL_IMAGES_HOOKS+ > > I'm a bit worried about this POST_INSTALL_IMAGES_HOOKS could be > mis-understood or mis-used... > Maybe it'd be worth to add few words saying that it's mainly for > bootloader purpose. Since the POST_INSTALL_IMAGES_HOOKS seems to be controversial, at least its presence in the documentation, I think it is not worth to document it. My point is the documentation should contain as much information as possible to safely hack Buildroot; so, avoid documenting things that may be mis-understood or mis-used. So Davide, could you repost the patch without the POST_INSTALL_IMAGES_HOOKS? BTW, this hook is used in cases as special as rare (only used once in the current Buildroot code base...). Also, after some talk on the IRC chan, Thomas P. may have a way to not use it where it is currently used. Regards,
diff --git a/docs/manual/adding-packages-generic.txt b/docs/manual/adding-packages-generic.txt index b05043a..a2f37ec 100644 --- a/docs/manual/adding-packages-generic.txt +++ b/docs/manual/adding-packages-generic.txt @@ -357,6 +357,9 @@ userful for generic packages. The following hook points are available: +* +LIBFOO_POST_DOWNLOAD_HOOKS+ +* +LIBFOO_POST_EXTRACT_HOOKS+ +* +LIBFOO_PRE_PATCH_HOOKS+ * +LIBFOO_POST_PATCH_HOOKS+ * +LIBFOO_PRE_CONFIGURE_HOOKS+ * +LIBFOO_POST_CONFIGURE_HOOKS+ @@ -364,6 +367,7 @@ The following hook points are available: * +LIBFOO_POST_INSTALL_HOOKS+ (for host packages only) * +LIBFOO_POST_INSTALL_STAGING_HOOKS+ (for target packages only) * +LIBFOO_POST_INSTALL_TARGET_HOOKS+ (for target packages only) +* +LIBFOO_POST_INSTALL_IMAGES_HOOKS+ * +LIBFOO_POST_LEGAL_INFO_HOOKS+ These variables are 'lists' of variable names containing actions to be