Bug#820119: restores original characters instead of taking care of every time numeric references coming up

2017-01-14 Thread Laura Arjona Reina 
Hi

El 13/01/17 a las 11:34, victory escribió:
> 
> first, it is stupid to blame about names which are valid.
> it is also stupid that taking care of each occurrences coming up.
> as pages are all utf-8 now, no need to keep such references,
> this patch restores original characters instead of numeric references
> 
> patch below:
> Index: english/international/l10n/scripts/gen-files.pl
> ===
> --- english/international/l10n/scripts/gen-files.pl   (revision 232)
> +++ english/international/l10n/scripts/gen-files.pl   (working copy)
> @@ -3,6 +3,7 @@
>  use strict;
>  use File::Path;
>  use Getopt::Long;
> +use Encode qw(encode);
>  
>  use lib ($0 =~ m|(.*)/|, $1 or ".") ."/../../../../Perl";
>  
> @@ -117,8 +118,7 @@
>  $name =~ s/\s*<.*//;
>  $name =~ s/&(?!#)/&/g;
>  $name =~ s/=\?.*?\?=//g;
> -# BREAK PERMITTED HERE (U+0082) is not allowed in HTML 4.01.
> -$name =~ s/(?:�*130;|�*82;|\N{U+0082})//ig;
> +$name =~ s/&#(\d+);/encode("UTF-8",chr($1))/ge;
>  $name = 'DDTP' if $name eq 'Debian Description Translation Project';
>  $name = '' if $name =~ m/\@/;
>  return $name;
> 
> 

Thanks for all the work in these and other validation/tidy issues in
the website.

I've done some tests and I'm afraid I cannot merge the patch yet.

Using perl to encode to UTF8 as you propose makes tidy happy, but
there is another script passed to the files, "validate", that produces
theses errors:

Line 10, character 12:  non SGML character number 130

If we use numeric entities, tidy complains for ‚ unless we
suppress the character as we do now.

For the emoji in translator name, "validate" complains in any case:

* Using numeric entities: with the current message received:

"128513" is not a character number in the document character set

* Encoding to UTF8 as the proposed patch:

Line 10, character 29:  non SGML character number 65533

I've produced two small files:

https://cosas.larjona.net/validate.utf8.html
https://cosas.larjona.net/validate.ncr.html

and passed the online validator in https://validator.w3.org/

I'll try to see if we can use https://validator.w3.org/source/ and get
better "tidy" and "validate" tools from there.

For now, I've fixed the comment in the gen-files.pl:

--- english/international/l10n/scripts/gen-files.pl 20 May 2016
21:15:45 -  1.97
+++ english/international/l10n/scripts/gen-files.pl 14 Jan 2017
12:41:06 -
@@ -117,7 +117,10 @@
 $name =~ s/\s*<.*//;
 $name =~ s/&(?!#)/&/g;
 $name =~ s/=\?.*?\?=//g;
-# BREAK PERMITTED HERE (U+0082) is not allowed in HTML 4.01.
+# BREAK PERMITTED HERE (U+0082) is allowed in HTML 4.01.
+# but the "tidy" tool that we use complains about them,
+# so we just remove those characters for now, until better
solution
+# see Bug #820119
 $name =~ s/(?:�*130;|�*82;|\N{U+0082})//ig;
 $name = 'DDTP' if $name eq 'Debian Description Translation
Project';
 $name = '' if $name =~ m/\@/;

Best regards
-- 
Laura Arjona Reina
https://wiki.debian.org/LauraArjona

Bug#820119: restores original characters instead of taking care of every time numeric references coming up

2017-01-14 Thread Laura Arjona Reina 
Hi again.

I think my conclusion is silly, I was considering encoding the whole string 
only.
But we can encode  the &000130 and leave the emoji in numeric entity.
Victory is right, I'll try to think clearer later and merge the patch today. 
(Now afk, sorry).


El 14 de enero de 2017 13:43:24 CET, Laura Arjona Reina  
escribió:
>Hi
>
>El 13/01/17 a las 11:34, victory escribió:
>> 
>> first, it is stupid to blame about names which are valid.
>> it is also stupid that taking care of each occurrences coming up.
>> as pages are all utf-8 now, no need to keep such references,
>> this patch restores original characters instead of numeric references
>> 
>> patch below:
>> Index: english/international/l10n/scripts/gen-files.pl
>> ===
>> --- english/international/l10n/scripts/gen-files.pl  (revision 232)
>> +++ english/international/l10n/scripts/gen-files.pl  (working copy)
>> @@ -3,6 +3,7 @@
>>  use strict;
>>  use File::Path;
>>  use Getopt::Long;
>> +use Encode qw(encode);
>>  
>>  use lib ($0 =~ m|(.*)/|, $1 or ".") ."/../../../../Perl";
>>  
>> @@ -117,8 +118,7 @@
>>  $name =~ s/\s*<.*//;
>>  $name =~ s/&(?!#)/&/g;
>>  $name =~ s/=\?.*?\?=//g;
>> -# BREAK PERMITTED HERE (U+0082) is not allowed in HTML 4.01.
>> -$name =~ s/(?:�*130;|�*82;|\N{U+0082})//ig;
>> +$name =~ s/&#(\d+);/encode("UTF-8",chr($1))/ge;
>>  $name = 'DDTP' if $name eq 'Debian Description Translation
>Project';
>>  $name = '' if $name =~ m/\@/;
>>  return $name;
>> 
>> 
>
>Thanks for all the work in these and other validation/tidy issues in
>the website.
>
>I've done some tests and I'm afraid I cannot merge the patch yet.
>
>Using perl to encode to UTF8 as you propose makes tidy happy, but
>there is another script passed to the files, "validate", that produces
>theses errors:
>
>Line 10, character 12:  non SGML character number 130
>
>If we use numeric entities, tidy complains for ‚ unless we
>suppress the character as we do now.
>
>For the emoji in translator name, "validate" complains in any case:
>
>* Using numeric entities: with the current message received:
>
>"128513" is not a character number in the document character set
>
>* Encoding to UTF8 as the proposed patch:
>
>Line 10, character 29:  non SGML character number 65533
>
>I've produced two small files:
>
>https://cosas.larjona.net/validate.utf8.html
>https://cosas.larjona.net/validate.ncr.html
>
>and passed the online validator in https://validator.w3.org/
>
>I'll try to see if we can use https://validator.w3.org/source/ and get
>better "tidy" and "validate" tools from there.
>
>For now, I've fixed the comment in the gen-files.pl:
>
>--- english/international/l10n/scripts/gen-files.pl 20 May 2016
>21:15:45 -  1.97
>+++ english/international/l10n/scripts/gen-files.pl 14 Jan 2017
>12:41:06 -
>@@ -117,7 +117,10 @@
> $name =~ s/\s*<.*//;
> $name =~ s/&(?!#)/&/g;
> $name =~ s/=\?.*?\?=//g;
>-# BREAK PERMITTED HERE (U+0082) is not allowed in HTML 4.01.
>+# BREAK PERMITTED HERE (U+0082) is allowed in HTML 4.01.
>+# but the "tidy" tool that we use complains about them,
>+# so we just remove those characters for now, until better
>solution
>+# see Bug #820119
> $name =~ s/(?:�*130;|�*82;|\N{U+0082})//ig;
> $name = 'DDTP' if $name eq 'Debian Description Translation
>Project';
> $name = '' if $name =~ m/\@/;
>
>Best regards

Laura Arjona Reina
https://wiki.debian.org/LauraArjona

Bug#851241: packages.debian.org: update hangs every now and then

2017-01-14 Thread Laura Arjona Reina 
Package: www.debian.org
User: www.debian@packages.debian.org
Usertags: packages

thanks

-- 
Laura Arjona Reina
https://wiki.debian.org/LauraArjona

Bug#804407: marked as done (www.debian.org: we need debian forums in spanish)

2017-01-14 Thread Debian Bug Tracking System 
Your message dated Sat, 14 Jan 2017 23:06:56 +
with message-id 
and subject line Debian WWW CVS commit by larjona fixes #804407
has caused the Debian Bug report #804407,
regarding www.debian.org: we need debian forums in spanish
to be marked as done.

This means that you claim that the problem has been dealt with.
If this is not the case it is now your responsibility to reopen the
Bug report if necessary, and/or fix the problem forthwith.

(NB: If you are a system administrator and have no idea what this
message is talking about, this may indicate a serious mail system
misconfiguration somewhere. Please contact ow...@bugs.debian.org
immediately.)


-- 
804407: http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=804407
Debian Bug Tracking System
Contact ow...@bugs.debian.org with problems
--- Begin Message ---
Package: www.debian.org
Severity: important

Dear Maintainer,

we need debian forums in spanish

i can work like moderator,
i have alot free time
It would be a pleasure to help with this

english:  http://forums.debian.net

Spanish:
es.forums.debian.net
or
http://forums.debian.net/es



-- System Information:
Debian Release: stretch/sid
  APT prefers unstable
  APT policy: (500, 'unstable'), (500, 'testing')
Architecture: i386 (i686)

Kernel: Linux 4.0.0-2-686-pae (SMP w/1 CPU core)
Locale: LANG=es_MX.UTF-8, LC_CTYPE=es_MX.UTF-8 (charmap=UTF-8)
Shell: /bin/sh linked to /bin/dash
Init: sysvinit (via /sbin/init)
--- End Message ---
--- Begin Message ---
This bug was closed by larjona in the webwml CVS repository:

https://www.debian.org/devel/website/using_cvs

Note that it might take some time until www.debian.org has been updated.

CVSROOT:/cvs/webwml
Module name:webwml
Changes by: larjona 17/01/14 23:06:56

Modified files:
swedish/devel/website: using_cvs.wml 

Log message:
Fix typos

Modified files:
english/misc   : related_links.wml 

Log message:
Added links to exdebian.org since it has grown in the last times since 
esdebian.org went down. Closes: #804407--- End Message ---

Bug#851439: packages.debian.org: Inaccurate message on package download page (right click needed in some browsers)

2017-01-14 Thread Laura Arjona Reina 
Package: www.debian.org
Severity: normal
User: www.debian@packages.debian.org
Usertags: packages

Resending to the BTS so it doesn't get lost.

Cheers

-- 
Laura Arjona Reina
https://wiki.debian.org/LauraArjona


 Mensaje reenviado 
Asunto: Inaccurate message on package download page
Resent-Date: Thu, 28 Jul 2016 15:03:13 + (UTC)
Resent-From: debian-www@lists.debian.org
Fecha: Thu, 28 Jul 2016 15:45:26 +0100
De: George Bateman 
Para: debian-www@lists.debian.org

Dear Debian Website mailing list,

At (eg.) https://packages.debian.org/wheezy/all/libtomcat7-java/download,
the following message is displayed:

>Note that in some browsers you will need to tell your browser you want the 
>file saved to a file. For example, in Firefox or Mozilla, you should hold the 
>Shift key when you click on the URL.

This isn't true any more; please could it be removed?

Thanks,

George Bateman.

Bug#851439: packages.debian.org: [PATCH] Inaccurate message on package download page (right click needed in some browsers)

2017-01-14 Thread Laura Arjona Reina 
Package: www.debian.org
Followup-For: Bug #851439

I'm attaching a patch that removes the paragraph about the need of right click
in some browsers.
I can commit it in some days, if nobody disagrees.
Cheers



-- System Information:
Debian Release: 8.7
  APT prefers stable-updates
  APT policy: (500, 'stable-updates'), (500, 'stable')
Architecture: i386 (i686)

Kernel: Linux 3.16.0-4-686-pae (SMP w/2 CPU cores)
Locale: LANG=es_ES.UTF-8, LC_CTYPE=es_ES.UTF-8 (charmap=UTF-8)
Shell: /bin/sh linked to /bin/dash
Init: systemd (via /run/systemd/system)
>From 8b537184c2f1ba7b122ba800ddd81a38e8c5ef24 Mon Sep 17 00:00:00 2001
From: Laura Arjona Reina 
Date: Sun, 15 Jan 2017 00:23:59 +0100
Subject: [PATCH] Remove old reference to need of right click in browsers to
 download. Closes: #851439

---
 templates/html/download.tmpl | 2 --
 1 file changed, 2 deletions(-)

diff --git a/templates/html/download.tmpl b/templates/html/download.tmpl
index 82509a8..b6c8e05 100644
--- a/templates/html/download.tmpl
+++ b/templates/html/download.tmpl
@@ -97,8 +97,6 @@ deb http://[% mirrors.$archive.europa.0 %] [% suite %] main [% section
 	port.name, organisation, port.name, port.url, port.name) %]
 [% END %]
 
-[% g('Note that in some browsers you will need to tell your browser you want the file saved to a file. For example, in Firefox or Mozilla, you should hold the Shift key when you click on the URL.') %]
-
 
 
 [% g('More information on %s:', filename.file) %]
-- 
2.1.4

Re: Fwd: Hexagonal sticker with the debian swirl

2017-01-14 Thread Laura Arjona Reina 
El 29/09/16 a las 08:59, Elena ``of Valhalla'' escribió:
> On 2016-09-28 at 20:32:40 +0200, Stéphane Blondon wrote:
>> Le 28/09/2016 à 10:07, Elena ``of Valhalla'' a écrit :
>>> In the IRC conversation it was also mentioned that the sticker could be
>>> then submitted to http://hexb.in/ : I would be happy to do it, but I
>>> would prefer it there was already a canonical location for the files to
>>> point to in the metadata json.
>> I don't if you posted it to hexb.in finally but the image is displayed
>> on this website (at the bottom of the logos).
> 
> yes, back in July I posted a second message saying that I found out that
> there was no need for an external url and submitted it, but forgot to
> mention it in yesterday's email.
> 
> (it's not always at the bottom, that page is shuffled at every reload.)
> 
Long overdue, I just committed the files, and the sticker should be
visible in https://www.debian.org/logos/ in the following hours.

Tomorrow we'll microblog about it.

Thanks Elena for the work, and sorry for the delay!

-- 
Laura Arjona Reina
https://wiki.debian.org/LauraArjona

Bug#659066: marked as done (www.debian.org: [fr] inconsistent capitalization of 'Développeurs', 'Contrat social' and 'Comité technique' in the Social Contract)

2017-01-14 Thread Debian Bug Tracking System 
Your message dated Sun, 15 Jan 2017 01:25:45 +0100
with message-id <783f91d3-dd34-f1ae-213a-7ae3a422e...@debian.org>
and subject line www.debian.org: [fr] inconsistent capitalization of 
'Développeurs', 'Contrat social' and 'Comité technique' in the Social Contract
has caused the Debian Bug report #659066,
regarding www.debian.org: [fr] inconsistent capitalization of 'Développeurs', 
'Contrat social' and 'Comité technique' in the Social Contract
to be marked as done.

This means that you claim that the problem has been dealt with.
If this is not the case it is now your responsibility to reopen the
Bug report if necessary, and/or fix the problem forthwith.

(NB: If you are a system administrator and have no idea what this
message is talking about, this may indicate a serious mail system
misconfiguration somewhere. Please contact ow...@bugs.debian.org
immediately.)


-- 
659066: http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=659066
Debian Bug Tracking System
Contact ow...@bugs.debian.org with problems
--- Begin Message ---
Package: www.debian.org
Severity: normal
Usertags: pca.it-l10n

Hi there!

Cc:ing the debian-l10n-french@ mailing list for comments and
doc-debian-fr@p.d.o because the bug is in this package.

While reading the French version [1] of the Social Contract for my talk
tomorrow at the Open Source Now 2012 [2], I discovered that there is an
inconsistency in the capitalization of the first letter of the word
'Développeurs': at §2.1 and §4.1 it is uppercase, while every other
times it is lowercase.  Similarly for 'Contrat social' at §4.1.5.2 and
'Comité technique' at §6.

[1] 
[2] 

Given how English terms have been translated (always with a lowercase
first letter), I would say that the use at §2.1, §4.1, §4.1.5.2 and §6
is wrong.

However, IMHO we should keep these words as in the English version,
i.e. with a uppercase first letter.  This is the situation in all the
other languages but Danish and Swedish.

Thx, bye,
Gismo / Luca

-- System Information:
Debian Release: wheezy/sid
  APT prefers unstable
  APT policy: (990, 'unstable'), (1, 'experimental')
Architecture: amd64 (x86_64)

Kernel: Linux 3.2.0-1-amd64 (SMP w/2 CPU cores)
Locale: LANG=en_US.UTF-8, LC_CTYPE=en_US.UTF-8 (charmap=UTF-8)
Shell: /bin/sh linked to /bin/dash


pgp8GjDjRLwyR.pgp
Description: PGP signature
--- End Message ---
--- Begin Message ---
Hi everybody
I've reviewed the current version of the constitution in french:

https://www.debian.org/devel/constitution.fr.html

And it seems that all the inconsistencies reported in this bug are
gone, except for the first mention of Comité technique in section 6.1.

Given that that mention is the one containing the link, I assume that
this bug is fixed and the mentioned capitalization is intentional.

Closing the bug in www.debian.org pseudopackage, thus, and CC'ing the
French team for the case they want to review themselves this issue in
their team.

Thanks
-- 
Laura Arjona Reina
https://wiki.debian.org/LauraArjona--- End Message ---

Bug#851458: not-enough-i18n'ed string on devel/website/stats/ pages

2017-01-14 Thread victory 

Package: www.debian.org
Severity: wishlist
Tags: patch l10n

I-have-known-since-5-years-ago bug, though, 
on current devel/website/stats/ pages,
a gettext string is not enough i18n'ed as it does not include its object;

w/ the patch applied, the string will become like
 "Created with "
instead of
 "Created with"
so we can translate it as
 " により生成しました"

patch below

Index: stattrans.pl
===
--- stattrans.pl(revision 240)
+++ stattrans.pl(working copy)
@@ -535,6 +535,11 @@
 if (open (HTML, ">$config{'htmldir'}/$l.wml")) {
printf HTML "#use wml::debian::template 
title=\"<:=\$trans{\$CUR_ISO_LANG}{%s}:>\"\n", $lang;
print HTML "#use wml::debian::toc\n";
+printf HTML qq|webwml-stattrans\n|,
+   alioth_cvs_view_url('stattrans.pl');
+printf HTML "\n";
+printf HTML "Created with 
\n"
+printf HTML "\n";
 print HTML "\n\n";
 $color = get_color ($percent_a{$lang});
 
@@ -655,10 +660,7 @@
 print HTML "\n";
 }
 
-print HTML
-'Created with webwml-stattrans' . "\n";
+print HTML "\n";
 close (HTML);
 } else {
 print "Can't open $config{'htmldir'}/$l.wml\n";
@@ -809,10 +811,7 @@
 print HTMLI "";
 print HTMLI "\n";
 
-print HTMLI
-'Created with webwml-stattrans' . "\n";
+print HTMLI "\n";
 close (HTMLI);
 
 print "done.\n" if ($config{'verbose'});


-- 
victory
no need to CC me :-)