Re: [PATCH] Spelling fixes
Hello Ville, thanks for your 3 patches. I applied them all. Are you interested in contributing more? I desperately need help as there are many things to work on. Cheers, On Tue, 24 Oct 2017, Ville Skyttä wrote: > --- > distro_tracker/auto_news/tests.py | 2 +- > distro_tracker/core/tests/tests_models.py | 2 +- > distro_tracker/core/tests/tests_tasks.py| 2 +- > distro_tracker/core/utils/email_messages.py | 2 +- > distro_tracker/core/utils/linkify.py| 2 +- > distro_tracker/mail/tests/tests_control.py | 4 ++-- > distro_tracker/mail/tests/tests_dispatch.py | 2 +- > distro_tracker/project/settings/defaults.py | 4 ++-- > distro_tracker/test/tests.py| 2 +- > distro_tracker/vendor/debian/tests.py | 2 +- > functional_tests/tests.py | 2 +- > 11 files changed, 13 insertions(+), 13 deletions(-) -- Raphaël Hertzog ◈ Debian Developer Support Debian LTS: https://www.freexian.com/services/debian-lts.html Learn to master Debian: https://debian-handbook.info/get/
Debian Jenkins IRC meeting @ 25.09.2017 18:00 (UTC)
Hey, Monthly Debian Jenkins Team Meeting will take place at #debian-qa on irc.oftc.net on 25.10.2017 18:00 (UTC) (4th Wednesday of the current month). You can check your local meeting time by running (thanks Paul!): date -d 'TZ="UTC" 2017-10-25 18:00' Logs from previous meetings can be found inside 'old meetings' section at [1]. Regards, T. Nitecki signature.asc Description: OpenPGP digital signature
Bug#879280: marked as done (fakeupstream.cgi: Use a 5xx HTTP status code in case of error)
Your message dated Tue, 24 Oct 2017 22:22:03 +
with message-id
and subject line qa.debian.org bug fixed in revision 3660
has caused the Debian Bug report #879280,
regarding fakeupstream.cgi: Use a 5xx HTTP status code in case of error
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.)
--
879280: https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=879280
Debian Bug Tracking System
Contact ow...@bugs.debian.org with problems
--- Begin Message ---
Package: qa.debian.org
Tags: patch
Dear fakeupstream.cgi maintainers,
HTTP calls to fakeupstream.cgi should return a 5xx HTTP status code in
case an error occurred while processing the request.
Luckily there is already in the code a single point of exit for all
errors. The following patch implements returning a 500 code in case of
error.
Bikeshedding time! :) I used a "500 Internal Server Error" status code.
"400 Bad Request" would also be OK. Actually every status code in the
4xx and 5xx range would be better than the current "200 OK".
Regards,
--- fakeupstream.cgi.orig 2017-10-21 14:24:26.493174836 +0200
+++ fakeupstream.cgi2017-10-21 15:25:01.940837493 +0200
@@ -71,7 +71,7 @@
{
chdir( "/" ); # for removal of vcs_tempdir
my $message = shift;
- print $q->header( "text/plain" );
+ print $q->header( "text/plain", "500 Internal Server Error" );
print "$message\n";
exit 0;
}
--
Gioele Barabucci
--- End Message ---
--- Begin Message ---
Version: 3660
This bug was closed by Mattia Rizzolo (mattia) in SVN revision 3660.
Note that it might take some time until the qa.debian.org code has
been updated and cronjobs have picked up changed data.
Commit message:
fakeupstream: return 500 in case or errors. (Closes: #879280)--- End Message ---
Bug#879278: marked as done (fakeupstream.cgi: Broken google-fonts upstream (probably svn/TLS certificate))
Your message dated Tue, 24 Oct 2017 22:38:13 + with message-id and subject line qa.debian.org bug fixed in revision 3661 has caused the Debian Bug report #879278, regarding fakeupstream.cgi: Broken google-fonts upstream (probably svn/TLS certificate) 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.) -- 879278: https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=879278 Debian Bug Tracking System Contact ow...@bugs.debian.org with problems --- Begin Message --- Package: qa.debian.org Severity: important Tags: patch The google-fonts upstream no longer works: it always answers that there is no version information about the requested font. See, for example https://qa.debian.org/cgi-bin/fakeupstream.cgi?upstream=google-fonts/ofl/pompiere According to some users of the #debian-qa IRC channel, the problem seems to be that svn does not know the all the TLS certificates in the GitHub's certificates chain. If this is the case, could you please fix the TLS configuration in svn or apply the following patch? --- fakeupstream.cgi.orig 2017-10-21 14:24:26.493174836 +0200 +++ fakeupstream.cgi2017-10-21 15:02:51.161463161 +0200 @@ -797,6 +797,8 @@ my @command = ( '/usr/bin/svn', + '--non-interactive', + '--trust-server-cert', 'cat', $svn_fontlog_url ); -- Gioele Barabucci --- End Message --- --- Begin Message --- Version: 3661 This bug was closed by Mattia Rizzolo (mattia) in SVN revision 3661. Note that it might take some time until the qa.debian.org code has been updated and cronjobs have picked up changed data. Commit message: fakeupstream: use LWP against github's export instead of SVN to gather google-fonts data Thanks: Gioele Barabucci for the patch Closes: #879278--- End Message ---
Re: [PATCH] Spelling fixes
On Tue, Oct 24, 2017 at 6:14 PM, Raphael Hertzog wrote: > Hello Ville, > > thanks for your 3 patches. I applied them all. Are you interested > in contributing more? > > I desperately need help as there are many things to work on. Sure, I think I can lend a hand every now and then. Is the list of things needing work available in public somewhere?
Re: [PATCH] Spelling fixes
On Wed, Oct 25, 2017 at 1:46 PM, Ville Skyttä wrote: > Sure, I think I can lend a hand every now and then. Is the list of > things needing work available in public somewhere? https://bugs.debian.org/tracker.debian.org -- bye, pabs https://wiki.debian.org/PaulWise
