Bug#1019872: remove unused scripts
Package: www.debian.org
Severity: normal
On IRC we had a discussion if the script
english/devel/website/stats/anoncvs-cors is still used. It seems that
more parts concerning the translations stats are not needed any more.
< Mrfai> Is this script used anywere:
english/devel/website/stats/anoncvs-cors ? I cannot find anything and would like to remove it
< Mrfai> There's only a link to http://webwml.alioth.debian.org/cgi-bin/anoncvs-cors in diffstat.js but I think this function is not used any more.
< pabs> `git grep` says stattrans.pl emits calls to setDiffstat in the onClick attribute of a <span> tag
< pabs> Mrfai: I think the onclick=setDiffstat can probably just be replaced with <a href= to salsa commit pages
< pabs> or maybe the compare URLs
< pabs> hmm, it wants diffstats not full patches...
< pabs> if the exact functionality is still wanted, then maybe the script could migrate to cgi.debian.org
< pabs> probably best to ask debian-i18n if the diffstat is useful
< Mrfai> pabs: But I cannot find any reference of this onlick in the html files.
< Mrfai> I did this:
< Mrfai> ...@wolkenstein:/srv/www.debian.org/www/international$ grep -r setDiffstat .
< Mrfai> And it found nothing
< pabs> I'm looking at stattrans again
< pabs> hmm, the Diffstat column is definitely present https://www.debian.org/devel/website/stats/en
< pabs> seems the diffstat link is only shown if "!defined $status_db{$lang}"
< pabs> which looks like the english/international/l10n/data/status.* files
< pabs> seems english/international/l10n/scripts/gen-files.pl creates those files
< pabs> those files do exist on wolkenstein
< pabs> they are symlinks to /srv/www.debian.org/cron/datafiles/status.*
< pabs> Sledge's a3fcc21a7c098056a557a095178aee9f752d6059 deleted a previous use of diffstat stuff, adding git command-lines instead, so it sounds like the web team
wants translators to manually do diffstats themselves, so I guess delete the diffstat column and all the supporting infrastructure
Reply to: