All of 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 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.