I tried compiling the package, it looks pretty good!

I have had some trouble in a few key places that still make the package
incomplete or unusable for me, here's an outline:

 1. lintian warnings
 2. no automatic database configuration
 3. MySQL GRANT too permissive
 4. checksum warnings
 5. sparkline generation fails
 6. empty pages

Let's go through this one at a time. :) If this gets fixed, I'd be happy
to sponsor this package. Let me know.

Lintian warnings
================

So far I have found the following lintian warnings:

W: piwik source: syntax-error-in-dep5-copyright line 14: Cannot parse line 
"Redistribution and use in source and binary forms, with or without "
W: piwik: embedded-javascript-library 
usr/share/piwik/libs/swfobject/swfobject.js

The former should be easy to cleanup.

The latter may be more problematic, but should probably be ignored if
the library isn't already in Debian.

Automatic database configuration
================================

Also, it seems to me you could leverage the dbconfig-common tools to
automatically create the database credentials for the site. I
understand, however, that this may be problematic since Piwik doesn't
provide a .sql file to load...

MySQL GRANT too permissive
==========================

Note that the documentation in README.Debian on how to grant database
access is way too permissive:

    GRANT ALL PRIVILEGES ON *.* TO 'piwikuser'@'localhost' WITH GRANT OPTION;

... will grant piwik access to *ALL* databases! The following should be
used instead:

    GRANT ALL PRIVILEGES ON piwik.* TO 'piwik'@'localhost' WITH GRANT OPTION;

This is also what is documented upstream:

http://piwik.org/docs/requirements/

Checksum warnings
=================

During the install, i see this warning:

 Le contrôle d'intégrité a échoué et a rapporté des erreurs. Cela est
 probablement du à un envoi de fichier partiel ou en échec des fichiers
 de Piwik. Vous devriez envoyer de nouveau les fichiers de Piwik et
 rafraichir cette page jusqu'à ce qu'elle n'affiche plus aucune
 erreur. Détails

(Sorry for the french, but this means that the file checksum failed,
probably because of the packaging.)

There are a bunch of files missing and others with the wrong checksum:

Différence de taille de fichier : /usr/share/piwik/core/HTMLPurifier.php 
(longueur attendue: 1120, rencontrée : 8831)
Fichier manquant : /usr/share/piwik/core/ViewDataTable/Sparkline.php
Fichier manquant : /usr/share/piwik/core/Visualization/Sparkline.php
Fichier manquant : /usr/share/piwik/js/LICENSE.txt
Fichier manquant : /usr/share/piwik/LEGALNOTICE
Fichier manquant : /usr/share/piwik/libs/cssmin/MIT-LICENSE.txt
Différence de taille de fichier : 
/usr/share/piwik/libs/HTML/QuickForm2/Element/Date.php (longueur attendue: 
35658, rencontrée : 42351)
Fichier manquant : /usr/share/piwik/libs/jqplot/MIT-LICENSE.txt
Différence de taille de fichier : 
/usr/share/piwik/libs/jquery/jquery.history.js (longueur attendue: 5592, 
rencontrée : 5922)
Différence de taille de fichier : /usr/share/piwik/libs/jquery/jquery.js 
(longueur attendue: 93868, rencontrée : 163855)
Différence de taille de fichier : /usr/share/piwik/libs/PclZip/pclzip.lib.php 
(longueur attendue: 196367, rencontrée : 203146)
Fichier manquant : /usr/share/piwik/libs/PEAR/LICENSE
Fichier manquant : /usr/share/piwik/libs/PiwikTracker/LICENSE.txt
Différence de taille de fichier : /usr/share/piwik/libs/Smarty/Smarty.class.php 
(longueur attendue: 63597, rencontrée : 63600)
Différence de taille de fichier : 
/usr/share/piwik/libs/Smarty/Smarty_Compiler.class.php (longueur attendue: 
94360, rencontrée : 94363)
Fichier manquant : /usr/share/piwik/libs/sparkline/lib/Sparkline.php
Fichier manquant : /usr/share/piwik/libs/swfobject/MIT-LICENSE.txt
Différence de taille de fichier : /usr/share/piwik/libs/Zend/Config/Json.php 
(longueur attendue: 8250, rencontrée : 17010)
Fichier manquant : /usr/share/piwik/libs/Zend/Config/Writer/Json.php
Fichier manquant : /usr/share/piwik/libs/Zend/Config/Writer/Yaml.php
Fichier manquant : /usr/share/piwik/libs/Zend/Config/Yaml.php
Fichier manquant : /usr/share/piwik/libs/Zend/LICENSE.txt
Fichier manquant : 
/usr/share/piwik/plugins/ExampleUI/images/icons-planet/LICENSE
Fichier manquant : /usr/share/piwik/plugins/SecurityInfo/LICENSE
Fichier manquant : /usr/share/piwik/README

("Fichier manquant" means "file missing" and "Différence de taille"
means "different size".)

Sparkline generation problem
============================

The sparklines seem to be broken, when viewing an image, i get this:

Fatal error: Class 'Piwik_ViewDataTable_Sparkline' not found in
/usr/share/piwik/core/ViewDataTable.php on line 229

This happens on the following URL for me:

http://localhost/piwik/index.php?module=MultiSites&action=getEvolutionGraph&period=day&date=2012-04-04,2012-05-03&evolutionBy=visits&columns=nb_visits&idSite=1&idsite=1&viewDataTable=sparkline

Empty pages
===========

In general, most pages in the UI are empty. This may be related to the
sparkline problem, but in effect, the piwik site is unusable: most pages
do not do anything when clicked (e.g. when i click on Visitors, the page
doesn't change).

There are no graphs in any page.

A.

-- 
Ils versent un pauvre miel sur leurs mots pourris et te parlent de pénurie
Et sur ta faim, sur tes amis, ils aiguisent leur appétit
                        - Richard Desjardins, La maison est ouverte

Attachment: pgpOSS5gieca2.pgp
Description: PGP signature

Reply via email to