diff mbox

[1/2] pkg-stats: add luarocks perl python virtual infrastructures

Message ID 1400066654-32196-1-git-send-email-fabio.porcedda@gmail.com
State Accepted
Commit a9463d24d47aea364ecc1c1ccb1aaf626d6e03c8
Headers show

Commit Message

Fabio Porcedda May 14, 2014, 11:24 a.m. UTC
Recongize the recently added infrastrucures: luarocks, perl, python,
virtual.

Signed-off-by: Fabio Porcedda <fabio.porcedda@gmail.com>
Cc: Jerzy Grzegorek <jerzy.grzegorek@trzebnica.net>
Cc: Luca Ceresoli <luca@lucaceresoli.net>
Cc: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
---
 support/scripts/pkg-stats | 71 +++++++++++++++++++++++++++++++++++++++++++++++
 1 file changed, 71 insertions(+)

Comments

Thomas Petazzoni May 14, 2014, 11:32 a.m. UTC | #1
Dear Fabio Porcedda,

On Wed, 14 May 2014 13:24:13 +0200, Fabio Porcedda wrote:
> Recongize the recently added infrastrucures: luarocks, perl, python,
> virtual.
> 
> Signed-off-by: Fabio Porcedda <fabio.porcedda@gmail.com>
> Cc: Jerzy Grzegorek <jerzy.grzegorek@trzebnica.net>
> Cc: Luca Ceresoli <luca@lucaceresoli.net>
> Cc: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
> ---
>  support/scripts/pkg-stats | 71 +++++++++++++++++++++++++++++++++++++++++++++++
>  1 file changed, 71 insertions(+)

Acked-by: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
Peter Korsgaard May 15, 2014, 9:38 p.m. UTC | #2
>>>>> "Fabio" == Fabio Porcedda <fabio.porcedda@gmail.com> writes:

 > Recongize the recently added infrastrucures: luarocks, perl, python,
 > virtual.

Fixed the typos above and committed both, thanks.
Thomas Petazzoni May 16, 2014, 7:31 a.m. UTC | #3
Dear Fabio Porcedda,

On Wed, 14 May 2014 13:24:13 +0200, Fabio Porcedda wrote:
> Recongize the recently added infrastrucures: luarocks, perl, python,
> virtual.
> 
> Signed-off-by: Fabio Porcedda <fabio.porcedda@gmail.com>
> Cc: Jerzy Grzegorek <jerzy.grzegorek@trzebnica.net>
> Cc: Luca Ceresoli <luca@lucaceresoli.net>
> Cc: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
> ---
>  support/scripts/pkg-stats | 71 +++++++++++++++++++++++++++++++++++++++++++++++
>  1 file changed, 71 insertions(+)

Today's output is now correct thanks to your patch:
http://autobuild.buildroot.org/stats/. And it now properly concludes
that there are 0 packages not using any infrastructure \o/

Thanks for having worked on updating this script!

Best regards,

Thomas
Fabio Porcedda May 16, 2014, 9:12 a.m. UTC | #4
On Fri, May 16, 2014 at 9:31 AM, Thomas Petazzoni
<thomas.petazzoni@free-electrons.com> wrote:
> Dear Fabio Porcedda,
>
> On Wed, 14 May 2014 13:24:13 +0200, Fabio Porcedda wrote:
>> Recongize the recently added infrastrucures: luarocks, perl, python,
>> virtual.
>>
>> Signed-off-by: Fabio Porcedda <fabio.porcedda@gmail.com>
>> Cc: Jerzy Grzegorek <jerzy.grzegorek@trzebnica.net>
>> Cc: Luca Ceresoli <luca@lucaceresoli.net>
>> Cc: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
>> ---
>>  support/scripts/pkg-stats | 71 +++++++++++++++++++++++++++++++++++++++++++++++
>>  1 file changed, 71 insertions(+)
>
> Today's output is now correct thanks to your patch:
> http://autobuild.buildroot.org/stats/. And it now properly concludes
> that there are 0 packages not using any infrastructure \o/

Yeah that's great, packages not using any infrastructure are
problematic for top level parallel make so I was glad to find that
there aren't such packages.

> Thanks for having worked on updating this script!

Best regards
Mike Zick May 16, 2014, 12:42 p.m. UTC | #5
On Fri, 16 May 2014 09:31:42 +0200
Thomas Petazzoni <thomas.petazzoni@free-electrons.com> wrote:

> Today's output is now correct thanks to your patch:
> http://autobuild.buildroot.org/stats/. And it now properly concludes
> that there are 0 packages not using any infrastructure \o/
>

Nice looking report.

A very tiny nit:
Pick a single spelling of license/licence in the summary section.

Mike
diff mbox

Patch

diff --git a/support/scripts/pkg-stats b/support/scripts/pkg-stats
index 4effa59..fd60d7d 100755
--- a/support/scripts/pkg-stats
+++ b/support/scripts/pkg-stats
@@ -72,6 +72,10 @@  td.lotsofpatches {
 
 autotools_packages=0
 cmake_packages=0
+luarocks_package=0
+perl_packages=0
+python_packages=0
+virtual_packages=0
 generic_packages=0
 manual_packages=0
 packages_with_licence=0
@@ -99,6 +103,10 @@  for i in $(find boot/ linux/ package/ -name '*.mk' | sort) ; do
 	$i = "package/x11r7/x11r7.mk" -o \
  	$i = "package/pkg-autotools.mk" -o \
  	$i = "package/pkg-cmake.mk" -o \
+	$i = "package/pkg-luarocks.mk" -o \
+	$i = "package/pkg-perl.mk" -o \
+	$i = "package/pkg-python.mk" -o \
+	$i = "package/pkg-virtual.mk" -o \
 	$i = "package/pkg-download.mk" -o \
  	$i = "package/pkg-generic.mk" -o \
  	$i = "package/pkg-utils.mk" ; then
@@ -123,6 +131,45 @@  for i in $(find boot/ linux/ package/ -name '*.mk' | sort) ; do
 	hastarget=1
     fi
 
+    if grep -E "\(host-luarocks-package\)" $i > /dev/null ; then
+	infratype="luarocks"
+	hashost=1
+    fi
+
+    if grep -E "\(luarocks-package\)" $i > /dev/null ; then
+	infratype="luarocks"
+	hastarget=1
+    fi
+
+    if grep -E "\(host-perl-package\)" $i > /dev/null ; then
+	infratype="perl"
+	hashost=1
+    fi
+
+    if grep -E "\(perl-package\)" $i > /dev/null ; then
+	infratype="perl"
+	hastarget=1
+    fi
+
+    if grep -E "\(host-python-package\)" $i > /dev/null ; then
+	infratype="python"
+	hashost=1
+    fi
+
+    if grep -E "\(python-package\)" $i > /dev/null ; then
+	infratype="python"
+	hastarget=1
+    fi
+    if grep -E "\(host-virtual-package\)" $i > /dev/null ; then
+	infratype="virtual"
+	hashost=1
+    fi
+
+    if grep -E "\(virtual-package\)" $i > /dev/null ; then
+	infratype="virtual"
+	hastarget=1
+    fi
+
     if grep -E "\(host-generic-package\)" $i > /dev/null ; then
 	infratype="generic"
 	hashost=1
@@ -156,6 +203,14 @@  for i in $(find boot/ linux/ package/ -name '*.mk' | sort) ; do
 	autotools_packages=$(($autotools_packages+1))
     elif [ ${infratype} = "cmake" ]; then
 	cmake_packages=$(($cmake_packages+1))
+    elif [ ${infratype} = "luarocks" ]; then
+	luarocks_packages=$(($luarocks_packages+1))
+    elif [ ${infratype} = "perl" ]; then
+	perl_packages=$(($perl_packages+1))
+    elif [ ${infratype} = "python" ]; then
+	python_packages=$(($python_packages+1))
+    elif [ ${infratype} = "virtual" ]; then
+	virtual_packages=$(($virtual_packages+1))
     elif [ ${infratype} = "generic" ]; then
 	generic_packages=$(($generic_packages+1))
     fi
@@ -244,6 +299,22 @@  echo "<td>Packages using the <i>autotools</i> infrastructure</td>"
 echo "<td>$autotools_packages</td>"
 echo "</tr>"
 echo "<tr>"
+echo "<td>Packages using the <i>luarocks</i> infrastructure</td>"
+echo "<td>$luarocks_packages</td>"
+echo "</tr>"
+echo "<tr>"
+echo "<td>Packages using the <i>perl</i> infrastructure</td>"
+echo "<td>$perl_packages</td>"
+echo "</tr>"
+echo "<tr>"
+echo "<td>Packages using the <i>python</i> infrastructure</td>"
+echo "<td>$python_packages</td>"
+echo "</tr>"
+echo "<tr>"
+echo "<td>Packages using the <i>virtual</i> infrastructure</td>"
+echo "<td>$virtual_packages</td>"
+echo "</tr>"
+echo "<tr>"
 echo "<td>Packages not using any infrastructure</td>"
 echo "<td>$manual_packages</td>"
 echo "</tr>"