Buildroot Archive on lore.kernel.org
 help / color / mirror / Atom feed
* [Buildroot] [PATCH 0/3] Config.in{,host}: check-package pass
@ 2018-02-14  5:04 Gaël PORTAY
  2018-02-14  5:04 ` [Buildroot] [PATCH 1/3] package/*/Config.in.host: fix help text check-package warnings Gaël PORTAY
                   ` (3 more replies)
  0 siblings, 4 replies; 6+ messages in thread
From: Gaël PORTAY @ 2018-02-14  5:04 UTC (permalink / raw)
  To: buildroot

Hi,

Here are some fixes about warning reported by check-package.

Regards,
Ga?l PORTAY (3):
  package/*/Config.in.host: fix help text check-package warnings
  gconf: Config.in: fix help text wrapping
  pure-ftpd: Config.in: fix help text wrapping

 package/gconf/Config.in                  | 6 +++---
 package/mtools/Config.in.host            | 8 ++++----
 package/omap-u-boot-utils/Config.in.host | 4 ++--
 package/pkgconf/Config.in.host           | 9 +++++----
 package/pure-ftpd/Config.in              | 4 ++--
 package/pwgen/Config.in.host             | 4 ++--
 package/qemu/Config.in.host              | 3 ++-
 package/tegrarcm/Config.in.host          | 4 ++--
 8 files changed, 22 insertions(+), 20 deletions(-)

-- 
2.15.1

^ permalink raw reply	[flat|nested] 6+ messages in thread

* [Buildroot] [PATCH 1/3] package/*/Config.in.host: fix help text check-package warnings
  2018-02-14  5:04 [Buildroot] [PATCH 0/3] Config.in{,host}: check-package pass Gaël PORTAY
@ 2018-02-14  5:04 ` Gaël PORTAY
  2018-02-14  5:04 ` [Buildroot] [PATCH 2/3] gconf: Config.in: fix help text wrapping Gaël PORTAY
                   ` (2 subsequent siblings)
  3 siblings, 0 replies; 6+ messages in thread
From: Gaël PORTAY @ 2018-02-14  5:04 UTC (permalink / raw)
  To: buildroot

This commit fixes the warnings reported by check-package on the help
text of all package Config.in.host files, related to the formatting of
the help text: should start with a tab, then 2 spaces, then at most 62
characters.

Signed-off-by: Ga?l PORTAY <gael.portay@savoirfairelinux.com>
---
 package/mtools/Config.in.host            | 8 ++++----
 package/omap-u-boot-utils/Config.in.host | 4 ++--
 package/pkgconf/Config.in.host           | 9 +++++----
 package/pwgen/Config.in.host             | 4 ++--
 package/qemu/Config.in.host              | 3 ++-
 package/tegrarcm/Config.in.host          | 4 ++--
 6 files changed, 17 insertions(+), 15 deletions(-)

diff --git a/package/mtools/Config.in.host b/package/mtools/Config.in.host
index 96b332b039..42cb6f80fb 100644
--- a/package/mtools/Config.in.host
+++ b/package/mtools/Config.in.host
@@ -1,9 +1,9 @@
 config BR2_PACKAGE_HOST_MTOOLS
 	bool "host mtools"
 	help
-	  Mtools is a collection of utilities to access MS-DOS disks from
-	  Unix without mounting them. It supports Win'95 style long file
-	  names, OS/2 Xdf disks and 2m disks (store up to 1992k on a high
-	  density 3 1/2 disk).
+	  Mtools is a collection of utilities to access MS-DOS disks
+	  from Unix without mounting them. It supports Win'95 style long
+	  file names, OS/2 Xdf disks and 2m disks (store up to 1992k on
+	  a high density 3 1/2 disk).
 
 	  http://www.gnu.org/software/mtools/
diff --git a/package/omap-u-boot-utils/Config.in.host b/package/omap-u-boot-utils/Config.in.host
index 87dcc06938..1549674f01 100644
--- a/package/omap-u-boot-utils/Config.in.host
+++ b/package/omap-u-boot-utils/Config.in.host
@@ -3,7 +3,7 @@ config BR2_PACKAGE_HOST_OMAP_U_BOOT_UTILS
 	depends on BR2_arm || BR2_armeb
 	help
 	  U-Boot Utilities for Texas Instrument's OMAP platforms.
-	  This is a set of tools to control U-Boot from scripts, generate
-	  OMAP-specific signed image files and more.
+	  This is a set of tools to control U-Boot from scripts,
+	  generate OMAP-specific signed image files and more.
 
 	  https://github.com/nmenon/omap-u-boot-utils
diff --git a/package/pkgconf/Config.in.host b/package/pkgconf/Config.in.host
index 437a7bf6f9..56836a5cbf 100644
--- a/package/pkgconf/Config.in.host
+++ b/package/pkgconf/Config.in.host
@@ -1,9 +1,10 @@
 config BR2_PACKAGE_HOST_PKGCONF
 	bool "host pkgconf"
 	help
-	  pkgconf is a program which helps to configure compiler and linker
-	  flags for development frameworks. It is similar to pkg-config,
-	  but was written from scratch in the summer of 2011 to replace
-	  pkg-config, which now needs itself to build itself
+	  pkgconf is a program which helps to configure compiler and
+	  linker flags for development frameworks. It is similar to
+	  pkg-config, but was written from scratch in the summer of
+	  2011 to replace pkg-config, which now needs itself to build
+	  itself
 
 	  https://github.com/pkgconf/pkgconf
diff --git a/package/pwgen/Config.in.host b/package/pwgen/Config.in.host
index 0c56851dcb..20b55f3fc0 100644
--- a/package/pwgen/Config.in.host
+++ b/package/pwgen/Config.in.host
@@ -1,7 +1,7 @@
 config BR2_PACKAGE_HOST_PWGEN
 	bool "host pwgen"
 	help
-	  Pwgen is a small, GPL'ed password generator which creates passwords
-	  which can be easily memorized by a human.
+	  Pwgen is a small, GPL'ed password generator which creates
+	  passwords which can be easily memorized by a human.
 
 	  http://sourceforge.net/projects/pwgen/
diff --git a/package/qemu/Config.in.host b/package/qemu/Config.in.host
index 71f38ef41f..f14bcec6d6 100644
--- a/package/qemu/Config.in.host
+++ b/package/qemu/Config.in.host
@@ -9,7 +9,8 @@ config BR2_PACKAGE_HOST_QEMU
 	select BR2_PACKAGE_HOST_QEMU_LINUX_USER_MODE \
 		if !BR2_PACKAGE_HOST_QEMU_SYSTEM_MODE
 	help
-	  QEMU is a generic and open source machine emulator and virtualizer.
+	  QEMU is a generic and open source machine emulator and
+	  virtualizer.
 
 	  This option builds an emulator for your selected architecture.
 
diff --git a/package/tegrarcm/Config.in.host b/package/tegrarcm/Config.in.host
index 47590fa127..d8cbb947d6 100644
--- a/package/tegrarcm/Config.in.host
+++ b/package/tegrarcm/Config.in.host
@@ -2,7 +2,7 @@ config BR2_PACKAGE_HOST_TEGRARCM
 	bool "host tegrarcm"
 	depends on BR2_arm || BR2_armeb
 	help
-	  This program is used to send code to a Tegra device in recovery
-	  mode.
+	  This program is used to send code to a Tegra device in
+	  recovery mode.
 
 	  https://github.com/NVIDIA/tegrarcm
-- 
2.15.1

^ permalink raw reply related	[flat|nested] 6+ messages in thread

* [Buildroot] [PATCH 2/3] gconf: Config.in: fix help text wrapping
  2018-02-14  5:04 [Buildroot] [PATCH 0/3] Config.in{,host}: check-package pass Gaël PORTAY
  2018-02-14  5:04 ` [Buildroot] [PATCH 1/3] package/*/Config.in.host: fix help text check-package warnings Gaël PORTAY
@ 2018-02-14  5:04 ` Gaël PORTAY
  2018-02-14  5:04 ` [Buildroot] [PATCH 3/3] pure-ftpd: " Gaël PORTAY
  2018-02-14 20:27 ` [Buildroot] [PATCH 0/3] Config.in{,host}: check-package pass Thomas Petazzoni
  3 siblings, 0 replies; 6+ messages in thread
From: Gaël PORTAY @ 2018-02-14  5:04 UTC (permalink / raw)
  To: buildroot

This commit fixes the warnings reported by check-package on the help
text of package Config.in file, related to the formatting of the help
text: should start with a tab, then 2 spaces, then at most 62
characters.

Signed-off-by: Ga?l PORTAY <gael.portay@savoirfairelinux.com>
---
 package/gconf/Config.in | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/package/gconf/Config.in b/package/gconf/Config.in
index a967d61644..a2371b3f23 100644
--- a/package/gconf/Config.in
+++ b/package/gconf/Config.in
@@ -9,9 +9,9 @@ config BR2_PACKAGE_GCONF
 	select BR2_PACKAGE_LIBGLIB2
 	select BR2_PACKAGE_LIBXML2
 	help
-	  GConf was a system used by the GNOME desktop environment for storing
-	  configuration settings for the desktop and applications. It is similar
-	  to the Windows Registry.
+	  GConf was a system used by the GNOME desktop environment for
+	  storing configuration settings for the desktop and
+	  applications. It is similar to the Windows Registry.
 
 	  https://projects.gnome.org/gconf/
 
-- 
2.15.1

^ permalink raw reply related	[flat|nested] 6+ messages in thread

* [Buildroot] [PATCH 3/3] pure-ftpd: Config.in: fix help text wrapping
  2018-02-14  5:04 [Buildroot] [PATCH 0/3] Config.in{,host}: check-package pass Gaël PORTAY
  2018-02-14  5:04 ` [Buildroot] [PATCH 1/3] package/*/Config.in.host: fix help text check-package warnings Gaël PORTAY
  2018-02-14  5:04 ` [Buildroot] [PATCH 2/3] gconf: Config.in: fix help text wrapping Gaël PORTAY
@ 2018-02-14  5:04 ` Gaël PORTAY
  2018-02-14 17:19   ` Sam Voss
  2018-02-14 20:27 ` [Buildroot] [PATCH 0/3] Config.in{,host}: check-package pass Thomas Petazzoni
  3 siblings, 1 reply; 6+ messages in thread
From: Gaël PORTAY @ 2018-02-14  5:04 UTC (permalink / raw)
  To: buildroot

This commit fixes the warnings reported by check-package on the help
text of package Config.in file, related to the formatting of the help
text: should start with a tab, then 2 spaces, then at most 62
characters.

Signed-off-by: Ga?l PORTAY <gael.portay@savoirfairelinux.com>
---
 package/pure-ftpd/Config.in | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/package/pure-ftpd/Config.in b/package/pure-ftpd/Config.in
index d03170b7cf..c060102650 100644
--- a/package/pure-ftpd/Config.in
+++ b/package/pure-ftpd/Config.in
@@ -33,7 +33,7 @@ config BR2_PACKAGE_PURE_FTPD_QUOTAS
 config BR2_PACKAGE_PURE_FTPD_UPLOADSCRIPT
 	bool "uploadscript"
 	help
-	  Enable pure-upload script. Automatically run an external program
-	  after a successful upload.
+	  Enable pure-upload script. Automatically run an external
+	  program after a successful upload.
 
 endif
-- 
2.15.1

^ permalink raw reply related	[flat|nested] 6+ messages in thread

* [Buildroot] [PATCH 3/3] pure-ftpd: Config.in: fix help text wrapping
  2018-02-14  5:04 ` [Buildroot] [PATCH 3/3] pure-ftpd: " Gaël PORTAY
@ 2018-02-14 17:19   ` Sam Voss
  0 siblings, 0 replies; 6+ messages in thread
From: Sam Voss @ 2018-02-14 17:19 UTC (permalink / raw)
  To: buildroot

On Tue, Feb 13, 2018 at 11:04 PM, Ga?l PORTAY
<gael.portay@savoirfairelinux.com> wrote:
> This commit fixes the warnings reported by check-package on the help
> text of package Config.in file, related to the formatting of the help
> text: should start with a tab, then 2 spaces, then at most 62
> characters.

I thought I checked this, sorry.

>
> Signed-off-by: Ga?l PORTAY <gael.portay@savoirfairelinux.com>

Acked-by: Sam Voss <sam.voss@rockwellcollins.com>

> ---
>  package/pure-ftpd/Config.in | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/package/pure-ftpd/Config.in b/package/pure-ftpd/Config.in
> index d03170b7cf..c060102650 100644
> --- a/package/pure-ftpd/Config.in
> +++ b/package/pure-ftpd/Config.in
> @@ -33,7 +33,7 @@ config BR2_PACKAGE_PURE_FTPD_QUOTAS
>  config BR2_PACKAGE_PURE_FTPD_UPLOADSCRIPT
>         bool "uploadscript"
>         help
> -         Enable pure-upload script. Automatically run an external program
> -         after a successful upload.
> +         Enable pure-upload script. Automatically run an external
> +         program after a successful upload.
>
>  endif
> --
> 2.15.1
>
> _______________________________________________
> buildroot mailing list
> buildroot at busybox.net
> http://lists.busybox.net/mailman/listinfo/buildroot

^ permalink raw reply	[flat|nested] 6+ messages in thread

* [Buildroot] [PATCH 0/3] Config.in{,host}: check-package pass
  2018-02-14  5:04 [Buildroot] [PATCH 0/3] Config.in{,host}: check-package pass Gaël PORTAY
                   ` (2 preceding siblings ...)
  2018-02-14  5:04 ` [Buildroot] [PATCH 3/3] pure-ftpd: " Gaël PORTAY
@ 2018-02-14 20:27 ` Thomas Petazzoni
  3 siblings, 0 replies; 6+ messages in thread
From: Thomas Petazzoni @ 2018-02-14 20:27 UTC (permalink / raw)
  To: buildroot

Hello,

On Wed, 14 Feb 2018 00:04:55 -0500, Ga?l PORTAY wrote:

> Ga?l PORTAY (3):
>   package/*/Config.in.host: fix help text check-package warnings
>   gconf: Config.in: fix help text wrapping
>   pure-ftpd: Config.in: fix help text wrapping

I've applied the patches to the master branch, thanks. However, I'd
like to finally find a solution to validate all those Config.in files
from our .gitlab-ci.yml.

The blocking problem is that package/Config.in and
package/x11r7/Config.in generate a lot of warnings, but we in fact want
to keep them as-is. See the thread at
http://patchwork.ozlabs.org/patch/849882/.

Best regards,

Thomas
-- 
Thomas Petazzoni, CTO, Bootlin (formerly Free Electrons)
Embedded Linux and Kernel engineering
http://bootlin.com

^ permalink raw reply	[flat|nested] 6+ messages in thread

end of thread, other threads:[~2018-02-14 20:27 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2018-02-14  5:04 [Buildroot] [PATCH 0/3] Config.in{,host}: check-package pass Gaël PORTAY
2018-02-14  5:04 ` [Buildroot] [PATCH 1/3] package/*/Config.in.host: fix help text check-package warnings Gaël PORTAY
2018-02-14  5:04 ` [Buildroot] [PATCH 2/3] gconf: Config.in: fix help text wrapping Gaël PORTAY
2018-02-14  5:04 ` [Buildroot] [PATCH 3/3] pure-ftpd: " Gaël PORTAY
2018-02-14 17:19   ` Sam Voss
2018-02-14 20:27 ` [Buildroot] [PATCH 0/3] Config.in{,host}: check-package pass Thomas Petazzoni

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox