Re: Koji build fails, scratch-build works

2015-11-29 Thread Mattia Verga
... my fault. The package name (and the .spec file) is set to "kpmcore", 
while the review request and PackageDB are set to "KPMcore"...


Should I retire in PackageDB and then redo the request package process? 
Can I use the same review request by renaming it or I must do a new one?


Il 29/11/2015 18:21, Mattia Verga ha scritto:
I'm trying to make the first build of an approved package (kpmcore), 
I've imported the .src.rpm, but the build in rawhide fails (without 
giving me a lot of information...):

http://koji.fedoraproject.org/koji/taskinfo?taskID=12012584

The strange is scratch-build works:
http://koji.fedoraproject.org/koji/taskinfo?taskID=12012451

What's wrong with that?
--
devel mailing list
devel@lists.fedoraproject.org
http://lists.fedoraproject.org/admin/lists/devel@lists.fedoraproject.org


--
devel mailing list
devel@lists.fedoraproject.org
http://lists.fedoraproject.org/admin/lists/devel@lists.fedoraproject.org


Re: Koji build fails, scratch-build works

2015-11-29 Thread Tom Hughes

On 29/11/15 18:18, Michael Schwendt wrote:

On Sun, 29 Nov 2015 19:10:25 +0100, Mattia Verga wrote:


Solved by changing .spec file name as set in PackageDB.

The developer himself uses both kpmcore and KPMcore in the project, so I
think there's no problem in adopting KPMcore as name in Fedora.

Sorry for the noise.


Both
   https://fedoraproject.org/wiki/Packaging:Guidelines#Spec_File_Naming
and
   https://fedoraproject.org/wiki/Packaging:NamingGuidelines#General_Naming
apply.

And it can be quite tiresome, if such issues come up during review.


Yes, as the reviewer in this case I would definitely have objected to a 
mixed case name on the basis of the guidelines.


Of course the spec file I was reviewing wasn't mixed case and I didn't 
notice that the bug title was different.


It always used to be the case that the request for a package to be added 
to pkgdb was rejected if the title didn't match, so has a bug been 
introduced there by the new pkgdb process?


Tom

--
Tom Hughes (t...@compton.nu)
http://compton.nu/
--
devel mailing list
devel@lists.fedoraproject.org
http://lists.fedoraproject.org/admin/lists/devel@lists.fedoraproject.org


Re: Package review skipped and straight to repos?

2015-11-29 Thread Zbigniew Jędrzejewski-Szmek
On Sun, Nov 29, 2015 at 09:53:40PM +, Sérgio Basto wrote:
> On Sáb, 2015-11-28 at 14:56 -0700, Kevin Fenzi wrote:
> > On Sat, 28 Nov 2015 21:11:48 +
> > James Hogarth  wrote:
> > 
> > > I was flicking through package review requests to see if anything
> > > jumped out as interesting when I saw this:
> > > 
> > > https://bugzilla.redhat.com/show_bug.cgi?id=1280422
> 
> You have one mini review [1] say fixed and next comment say package
> request has been approved. Don't forget now packages requests are
> made directly with pkgdb [2] , so I think you are overreact ,
> especially if you look for emails. 

Package review is public and happens in the review bug. We also
require (in the sense of having a strong custom, maybe even if it's
not written anywhere explicitly), a checklist style review. This allows
other people to know that not only the reviewer acked the package,
but that they did it with enough diligence.

There's no sign of the review process here.

Zbyszek
--
devel mailing list
devel@lists.fedoraproject.org
http://lists.fedoraproject.org/admin/lists/devel@lists.fedoraproject.org


Koji build fails, scratch-build works

2015-11-29 Thread Mattia Verga
I'm trying to make the first build of an approved package (kpmcore), 
I've imported the .src.rpm, but the build in rawhide fails (without 
giving me a lot of information...):

http://koji.fedoraproject.org/koji/taskinfo?taskID=12012584

The strange is scratch-build works:
http://koji.fedoraproject.org/koji/taskinfo?taskID=12012451

What's wrong with that?
--
devel mailing list
devel@lists.fedoraproject.org
http://lists.fedoraproject.org/admin/lists/devel@lists.fedoraproject.org


[Bug 1277233] Please branch and build for EPEL 7

2015-11-29 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1277233



--- Comment #6 from Jon Ciesla  ---
Thanks!

-- 
You are receiving this mail because:
You are on the CC list for the bug.
--
Fedora Extras Perl SIG
http://www.fedoraproject.org/wiki/Extras/SIGs/Perl
perl-devel mailing list
perl-devel@lists.fedoraproject.org
http://lists.fedoraproject.org/admin/lists/perl-devel@lists.fedoraproject.org


Re: Can Koji handle a soname change and a self-dependency?

2015-11-29 Thread Kevin Fenzi
On Sun, 29 Nov 2015 00:35:37 +0100
Björn Persson  wrote:

> Is there a way to deal with the following situation in Koji?
> 
> · Build tool B has a build-time dependency on itself.
> · B is linked to library L version 1.
> · L gets upgraded to version 2, which changes its soname.
> 
> B needs to be rebuilt to link to libL.so.2, but building B requires a
> working B, which requires libL.so.1 because it hasn't been rebuilt
> yet.
> 
> As I understand it, when the L-2 package goes into the buildroot it
> immediately replaces L-1. Is there a way to keep L-1 available until B
> has been rebuilt?

Nope.

> Is the answer to link B statically?

Nope.

The usual way to handle this is to add a compat version of the library
(so.1) to L, rebuild B, then drop the compat library, ie just keep it
for one build to rebuild B.

I think for example libpng has used this in the past if you want an
example. 

kevin





pgpShDClqAOY7.pgp
Description: OpenPGP digital signature
--
devel mailing list
devel@lists.fedoraproject.org
http://lists.fedoraproject.org/admin/lists/devel@lists.fedoraproject.org

Re: Package review skipped and straight to repos?

2015-11-29 Thread Sérgio Basto
On Sáb, 2015-11-28 at 14:56 -0700, Kevin Fenzi wrote:
> On Sat, 28 Nov 2015 21:11:48 +
> James Hogarth  wrote:
> 
> > I was flicking through package review requests to see if anything
> > jumped out as interesting when I saw this:
> > 
> > https://bugzilla.redhat.com/show_bug.cgi?id=1280422

You have one mini review [1] say fixed and next comment say package
request has been approved. Don't forget now packages requests are
made directly with pkgdb [2] , so I think you are overreact ,
especially if you look for emails. 

[1] https://bugzilla.redhat.com/show_bug.cgi?id=1280422#c2
[2] https://admin.fedoraproject.org/pkgdb/

> > 
> > Thought I'd take a look as I hadn't had time to review it when it
> > first appeared and the requester had fixed the github breaking
> > fedora-review issue ...
> > 
> > I was very surprised to see :
> > 
> > Package request has been approved:
> > https://admin.fedoraproject.org/pkgdb/package/rpg
> > 
> > Given that there is no assignee, no review and no flags on the bug
> > was
> > there just a mistake in an automated process or has policy changed
> > on
> > actually needing a review?
> 
> No, there's no policy change. Looks like some kind of mistake in
> processing. 
> 
> I added a comment on the bug.
> 
> It would be best if it got reviewed and approved and then imported,
> but
> if not, I will go and remove it. It should not be in the collection
> without a review. 
> 
> kevin
> --
> devel mailing list
> devel@lists.fedoraproject.org
> http://lists.fedoraproject.org/admin/lists/devel@lists.fedoraproject.
> org
-- 
Sérgio M. B.

--
devel mailing list
devel@lists.fedoraproject.org
http://lists.fedoraproject.org/admin/lists/devel@lists.fedoraproject.org


Re: Koji build fails, scratch-build works

2015-11-29 Thread Mattia Verga

Solved by changing .spec file name as set in PackageDB.

The developer himself uses both kpmcore and KPMcore in the project, so I 
think there's no problem in adopting KPMcore as name in Fedora.


Sorry for the noise.
--
devel mailing list
devel@lists.fedoraproject.org
http://lists.fedoraproject.org/admin/lists/devel@lists.fedoraproject.org


eseyman uploaded Hijk-0.26.tar.gz for perl-Hijk

2015-11-29 Thread notifications
5c2c9e2f27c406e952c415decdadfd7e  Hijk-0.26.tar.gz

http://pkgs.fedoraproject.org/lookaside/pkgs/perl-Hijk/Hijk-0.26.tar.gz/md5/5c2c9e2f27c406e952c415decdadfd7e/Hijk-0.26.tar.gz
--
Fedora Extras Perl SIG
http://www.fedoraproject.org/wiki/Extras/SIGs/Perl
perl-devel mailing list
perl-devel@lists.fedoraproject.org
http://lists.fedoraproject.org/admin/lists/perl-devel@lists.fedoraproject.org


eseyman pushed to perl-Hijk (master). "Update to 0.26"

2015-11-29 Thread notifications
From fe9fca5b4c2ba55f37006d041ac5566df88db3a6 Mon Sep 17 00:00:00 2001
From: Emmanuel Seyman 
Date: Sun, 29 Nov 2015 12:44:31 +0100
Subject: Update to 0.26

---
 .gitignore |  1 +
 perl-Hijk.spec | 14 +++---
 sources|  2 +-
 3 files changed, 9 insertions(+), 8 deletions(-)

diff --git a/.gitignore b/.gitignore
index ba4c4ba..37ccecc 100644
--- a/.gitignore
+++ b/.gitignore
@@ -7,3 +7,4 @@
 /Hijk-0.20.tar.gz
 /Hijk-0.22.tar.gz
 /Hijk-0.24.tar.gz
+/Hijk-0.26.tar.gz
diff --git a/perl-Hijk.spec b/perl-Hijk.spec
index f7c9ebc..bd8d99a 100644
--- a/perl-Hijk.spec
+++ b/perl-Hijk.spec
@@ -1,6 +1,6 @@
 Name:   perl-Hijk
-Version:0.24
-Release:2%{?dist}
+Version:0.26
+Release:1%{?dist}
 Summary:Specialized HTTP client
 License:MIT
 
@@ -50,15 +50,11 @@ rather then web servers.
 %setup -q -n Hijk-%{version}
 
 %build
-%{__perl} Makefile.PL INSTALLDIRS=vendor
+%{__perl} Makefile.PL INSTALLDIRS=vendor NO_PACKLIST=1
 make %{?_smp_mflags}
 
 %install
 make pure_install DESTDIR=$RPM_BUILD_ROOT
-
-find $RPM_BUILD_ROOT -type f -name .packlist -exec rm -f {} \;
-find $RPM_BUILD_ROOT -depth -type d -exec rmdir {} 2>/dev/null \;
-
 %{_fixperms} $RPM_BUILD_ROOT/*
 
 %check
@@ -71,6 +67,10 @@ make test
 %{_mandir}/man3/Hijk*
 
 %changelog
+* Sun Nov 29 2015 Emmanuel Seyman  - 0.26-1
+- Update to 0.26
+- Use NO_PACKLIST=1 when creating Makefile
+
 * Tue Aug 11 2015 Petr Šabata  - 0.24-2
 - Prevent FTBFS by correcting the build time dependency list
 
diff --git a/sources b/sources
index 5db41c5..18f3f6c 100644
--- a/sources
+++ b/sources
@@ -1 +1 @@
-d414666940b95bb212eefe38d4d969e5  Hijk-0.24.tar.gz
+5c2c9e2f27c406e952c415decdadfd7e  Hijk-0.26.tar.gz
-- 
cgit v0.11.2



http://pkgs.fedoraproject.org/cgit/perl-Hijk.git/commit/?h=master=fe9fca5b4c2ba55f37006d041ac5566df88db3a6
--
Fedora Extras Perl SIG
http://www.fedoraproject.org/wiki/Extras/SIGs/Perl
perl-devel mailing list
perl-devel@lists.fedoraproject.org
http://lists.fedoraproject.org/admin/lists/perl-devel@lists.fedoraproject.org

eseyman uploaded Mojolicious-Plugin-CHI-0.12.tar.gz for perl-Mojolicious-Plugin-CHI

2015-11-29 Thread notifications
cf91c2e1306353647fc7a9fd8b881373  Mojolicious-Plugin-CHI-0.12.tar.gz

http://pkgs.fedoraproject.org/lookaside/pkgs/perl-Mojolicious-Plugin-CHI/Mojolicious-Plugin-CHI-0.12.tar.gz/md5/cf91c2e1306353647fc7a9fd8b881373/Mojolicious-Plugin-CHI-0.12.tar.gz
--
Fedora Extras Perl SIG
http://www.fedoraproject.org/wiki/Extras/SIGs/Perl
perl-devel mailing list
perl-devel@lists.fedoraproject.org
http://lists.fedoraproject.org/admin/lists/perl-devel@lists.fedoraproject.org


eseyman pushed to perl-Mojolicious-Plugin-CHI (master). "Update to 0.12"

2015-11-29 Thread notifications
>From f0efac54c89cd068b5ae0e1517f5982fabb5375b Mon Sep 17 00:00:00 2001
From: Emmanuel Seyman 
Date: Sun, 29 Nov 2015 12:53:20 +0100
Subject: Update to 0.12

---
 .gitignore   | 1 +
 perl-Mojolicious-Plugin-CHI.spec | 7 +--
 sources  | 2 +-
 3 files changed, 7 insertions(+), 3 deletions(-)

diff --git a/.gitignore b/.gitignore
index 6ba1525..1427e3a 100644
--- a/.gitignore
+++ b/.gitignore
@@ -1 +1,2 @@
 /Mojolicious-Plugin-CHI-0.11.tar.gz
+/Mojolicious-Plugin-CHI-0.12.tar.gz
diff --git a/perl-Mojolicious-Plugin-CHI.spec b/perl-Mojolicious-Plugin-CHI.spec
index df49cd4..5e4aabf 100644
--- a/perl-Mojolicious-Plugin-CHI.spec
+++ b/perl-Mojolicious-Plugin-CHI.spec
@@ -1,6 +1,6 @@
 Name:   perl-Mojolicious-Plugin-CHI
-Version:0.11
-Release:4%{?dist}
+Version:0.12
+Release:1%{?dist}
 Summary:Use CHI Caches in Mojolicious
 License:Artistic 2.0
 
@@ -58,6 +58,9 @@ make test
 %{_mandir}/man3/*
 
 %changelog
+* Sun Nov 29 2015 Emmanuel Seyman  - 0.12-1
+- Update to 0.12
+
 * Thu Jun 18 2015 Fedora Release Engineering  
- 0.11-4
 - Rebuilt for https://fedoraproject.org/wiki/Fedora_23_Mass_Rebuild
 
diff --git a/sources b/sources
index ad27f20..8d6389b 100644
--- a/sources
+++ b/sources
@@ -1 +1 @@
-7bcbb11915e5ad970da2b3eec5f5c764  Mojolicious-Plugin-CHI-0.11.tar.gz
+cf91c2e1306353647fc7a9fd8b881373  Mojolicious-Plugin-CHI-0.12.tar.gz
-- 
cgit v0.11.2



http://pkgs.fedoraproject.org/cgit/perl-Mojolicious-Plugin-CHI.git/commit/?h=master=f0efac54c89cd068b5ae0e1517f5982fabb5375b
--
Fedora Extras Perl SIG
http://www.fedoraproject.org/wiki/Extras/SIGs/Perl
perl-devel mailing list
perl-devel@lists.fedoraproject.org
http://lists.fedoraproject.org/admin/lists/perl-devel@lists.fedoraproject.org


[Bug 1285570] perl-Hijk-0.26 is available

2015-11-29 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1285570



--- Comment #3 from Upstream Release Monitoring 
 ---
eseyman's perl-Hijk-0.26-1.fc24 completed
http://koji.fedoraproject.org/koji/buildinfo?buildID=702136

-- 
You are receiving this mail because:
You are on the CC list for the bug.
--
Fedora Extras Perl SIG
http://www.fedoraproject.org/wiki/Extras/SIGs/Perl
perl-devel mailing list
perl-devel@lists.fedoraproject.org
http://lists.fedoraproject.org/admin/lists/perl-devel@lists.fedoraproject.org


rawhide report: 20151129 changes

2015-11-29 Thread Fedora Rawhide Report
Compose started at Sun Nov 29 05:15:02 UTC 2015
Broken deps for i386
--
[3Depict]
3Depict-0.0.18-2.fc23.i686 requires libmgl.so.7.3.0
[IQmol]
IQmol-2.3.0-9.fc24.i686 requires libboost_serialization.so.1.58.0
IQmol-2.3.0-9.fc24.i686 requires libboost_iostreams.so.1.58.0
IQmol-2.3.0-9.fc24.i686 requires libOpenMeshCore.so.3.2
[alliance]
alliance-5.0-40.20090901snap.fc22.i686 requires libXm.so.2
[ardour]
ardour-2.8.16-15.fc23.i686 requires libaubio.so.2
[ardour2]
ardour2-2.8.16-18.fc24.i686 requires libaubio.so.2
[eclipse-jbosstools]
eclipse-jbosstools-as-4.2.2-1.fc22.noarch requires 
osgi(org.eclipse.tm.terminal)
[fawkes]
fawkes-core-0.5.0-26.fc24.i686 requires libmicrohttpd.so.10
fawkes-plugin-player-0.5.0-26.fc24.i686 requires libgeos-3.4.2.so
fawkes-plugin-xmlrpc-0.5.0-26.fc24.i686 requires libmicrohttpd.so.10
[fedfind]
fedfind-1.6.2-2.fc24.noarch requires python2-cached_property
[gnash]
1:gnash-0.8.10-19.fc24.i686 requires libboost_thread.so.1.58.0
1:gnash-0.8.10-19.fc24.i686 requires libboost_system.so.1.58.0
1:gnash-0.8.10-19.fc24.i686 requires libboost_serialization.so.1.58.0
1:gnash-0.8.10-19.fc24.i686 requires libboost_program_options.so.1.58.0
1:gnash-0.8.10-19.fc24.i686 requires libboost_iostreams.so.1.58.0
1:gnash-0.8.10-19.fc24.i686 requires libboost_date_time.so.1.58.0
1:gnash-cygnal-0.8.10-19.fc24.i686 requires libboost_thread.so.1.58.0
1:gnash-cygnal-0.8.10-19.fc24.i686 requires libboost_system.so.1.58.0
1:gnash-cygnal-0.8.10-19.fc24.i686 requires 
libboost_serialization.so.1.58.0
1:gnash-cygnal-0.8.10-19.fc24.i686 requires 
libboost_program_options.so.1.58.0
1:gnash-cygnal-0.8.10-19.fc24.i686 requires libboost_iostreams.so.1.58.0
1:gnash-cygnal-0.8.10-19.fc24.i686 requires libboost_date_time.so.1.58.0
1:gnash-extension-dejagnu-0.8.10-19.fc24.i686 requires 
libboost_thread.so.1.58.0
1:gnash-extension-dejagnu-0.8.10-19.fc24.i686 requires 
libboost_system.so.1.58.0
1:gnash-extension-dejagnu-0.8.10-19.fc24.i686 requires 
libboost_program_options.so.1.58.0
1:gnash-extension-dejagnu-0.8.10-19.fc24.i686 requires 
libboost_iostreams.so.1.58.0
1:gnash-extension-fileio-0.8.10-19.fc24.i686 requires 
libboost_thread.so.1.58.0
1:gnash-extension-fileio-0.8.10-19.fc24.i686 requires 
libboost_system.so.1.58.0
1:gnash-extension-fileio-0.8.10-19.fc24.i686 requires 
libboost_program_options.so.1.58.0
1:gnash-extension-fileio-0.8.10-19.fc24.i686 requires 
libboost_iostreams.so.1.58.0
1:gnash-extension-lirc-0.8.10-19.fc24.i686 requires 
libboost_thread.so.1.58.0
1:gnash-extension-lirc-0.8.10-19.fc24.i686 requires 
libboost_system.so.1.58.0
1:gnash-extension-lirc-0.8.10-19.fc24.i686 requires 
libboost_program_options.so.1.58.0
1:gnash-extension-lirc-0.8.10-19.fc24.i686 requires 
libboost_iostreams.so.1.58.0
1:gnash-extension-mysql-0.8.10-19.fc24.i686 requires 
libboost_thread.so.1.58.0
1:gnash-extension-mysql-0.8.10-19.fc24.i686 requires 
libboost_system.so.1.58.0
1:gnash-extension-mysql-0.8.10-19.fc24.i686 requires 
libboost_program_options.so.1.58.0
1:gnash-extension-mysql-0.8.10-19.fc24.i686 requires 
libboost_iostreams.so.1.58.0
1:gnash-klash-0.8.10-19.fc24.i686 requires libboost_system.so.1.58.0
1:gnash-klash-0.8.10-19.fc24.i686 requires 
libboost_program_options.so.1.58.0
1:gnash-plugin-0.8.10-19.fc24.i686 requires libboost_iostreams.so.1.58.0
1:python-gnash-0.8.10-19.fc24.i686 requires libboost_thread.so.1.58.0
1:python-gnash-0.8.10-19.fc24.i686 requires libboost_system.so.1.58.0
1:python-gnash-0.8.10-19.fc24.i686 requires 
libboost_program_options.so.1.58.0
1:python-gnash-0.8.10-19.fc24.i686 requires libboost_iostreams.so.1.58.0
[golang-github-kraman-libcontainer]
golang-github-kraman-libcontainer-devel-0-0.4.gitd700e5b.fc24.noarch 
requires golang(github.com/docker/docker/pkg/netlink)
[golang-github-kubernetes-heapster]
golang-github-kubernetes-heapster-devel-0.16.1-1.fc24.noarch requires 
golang(github.com/google/cadvisor/info/v1)
golang-github-kubernetes-heapster-devel-0.16.1-1.fc24.noarch requires 
golang(github.com/google/cadvisor/client)
golang-github-kubernetes-heapster-devel-0.16.1-1.fc24.noarch requires 
golang(github.com/coreos/fleet/schema)
golang-github-kubernetes-heapster-devel-0.16.1-1.fc24.noarch requires 
golang(github.com/coreos/fleet/registry)
golang-github-kubernetes-heapster-devel-0.16.1-1.fc24.noarch requires 
golang(github.com/coreos/fleet/pkg)
golang-github-kubernetes-heapster-devel-0.16.1-1.fc24.noarch requires 
golang(github.com/coreos/fleet/machine)

eseyman uploaded Mojolicious-6.33.tar.gz for perl-Mojolicious

2015-11-29 Thread notifications
ca045d6633078361fd7b1f1e2d316852  Mojolicious-6.33.tar.gz

http://pkgs.fedoraproject.org/lookaside/pkgs/perl-Mojolicious/Mojolicious-6.33.tar.gz/md5/ca045d6633078361fd7b1f1e2d316852/Mojolicious-6.33.tar.gz
--
Fedora Extras Perl SIG
http://www.fedoraproject.org/wiki/Extras/SIGs/Perl
perl-devel mailing list
perl-devel@lists.fedoraproject.org
http://lists.fedoraproject.org/admin/lists/perl-devel@lists.fedoraproject.org


eseyman pushed to perl-Mojolicious (master). "Update to 6.33"

2015-11-29 Thread notifications
>From d1f4defc72af2815f637d241b8d9509aca57b7a0 Mon Sep 17 00:00:00 2001
From: Emmanuel Seyman 
Date: Sun, 29 Nov 2015 12:51:31 +0100
Subject: Update to 6.33

---
 .gitignore| 1 +
 perl-Mojolicious.spec | 7 +--
 sources   | 2 +-
 3 files changed, 7 insertions(+), 3 deletions(-)

diff --git a/.gitignore b/.gitignore
index 8fe88d6..8bf6560 100644
--- a/.gitignore
+++ b/.gitignore
@@ -183,3 +183,4 @@ Mojolicious-0.26.tar.gz
 /Mojolicious-6.24.tar.gz
 /Mojolicious-6.27.tar.gz
 /Mojolicious-6.31.tar.gz
+/Mojolicious-6.33.tar.gz
diff --git a/perl-Mojolicious.spec b/perl-Mojolicious.spec
index 4a721f4..b1e1652 100644
--- a/perl-Mojolicious.spec
+++ b/perl-Mojolicious.spec
@@ -1,5 +1,5 @@
 Name:   perl-Mojolicious
-Version:6.31
+Version:6.33
 Release:1%{?dist}
 Summary:A next generation web framework for Perl
 License:Artistic 2.0
@@ -10,7 +10,7 @@ Source0:
http://search.cpan.org/CPAN/authors/id/S/SR/SRI/Mojolicious-%{ve
 BuildArch:  noarch
 BuildRequires:  perl >= 0:5.010001
 BuildRequires:  perl(Compress::Raw::Zlib)
-BuildRequires:  perl(ExtUtils::MakeMaker)
+BuildRequires:  perl(ExtUtils::MakeMaker) >= 6.76
 BuildRequires:  perl(Data::Dumper)
 BuildRequires:  perl(Digest::MD5)
 BuildRequires:  perl(Digest::SHA)
@@ -65,6 +65,9 @@ make test
 %{_mandir}/man3/*
 
 %changelog
+* Sun Nov 29 2015 Emmanuel Seyman  - 6.33-1
+- Update to 6.33
+
 * Sun Nov 15 2015 Emmanuel Seyman  - 6.31-1
 - Update to 6.31
 
diff --git a/sources b/sources
index 65205ef..a0cd699 100644
--- a/sources
+++ b/sources
@@ -1 +1 @@
-fe84256d973e8374e48f987d666027dd  Mojolicious-6.31.tar.gz
+ca045d6633078361fd7b1f1e2d316852  Mojolicious-6.33.tar.gz
-- 
cgit v0.11.2



http://pkgs.fedoraproject.org/cgit/perl-Mojolicious.git/commit/?h=master=d1f4defc72af2815f637d241b8d9509aca57b7a0
--
Fedora Extras Perl SIG
http://www.fedoraproject.org/wiki/Extras/SIGs/Perl
perl-devel mailing list
perl-devel@lists.fedoraproject.org
http://lists.fedoraproject.org/admin/lists/perl-devel@lists.fedoraproject.org


[Bug 1283540] perl-Mojolicious-Plugin-CHI-0.12 is available

2015-11-29 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1283540



--- Comment #3 from Upstream Release Monitoring 
 ---
eseyman's perl-Mojolicious-Plugin-CHI-0.12-1.fc24 completed
http://koji.fedoraproject.org/koji/buildinfo?buildID=702137

-- 
You are receiving this mail because:
You are on the CC list for the bug.
--
Fedora Extras Perl SIG
http://www.fedoraproject.org/wiki/Extras/SIGs/Perl
perl-devel mailing list
perl-devel@lists.fedoraproject.org
http://lists.fedoraproject.org/admin/lists/perl-devel@lists.fedoraproject.org


[Bug 1283442] perl-Mojolicious-6.33 is available

2015-11-29 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1283442



--- Comment #6 from Upstream Release Monitoring 
 ---
eseyman's perl-Mojolicious-6.33-1.fc24 completed
http://koji.fedoraproject.org/koji/buildinfo?buildID=702139

-- 
You are receiving this mail because:
You are on the CC list for the bug.
--
Fedora Extras Perl SIG
http://www.fedoraproject.org/wiki/Extras/SIGs/Perl
perl-devel mailing list
perl-devel@lists.fedoraproject.org
http://lists.fedoraproject.org/admin/lists/perl-devel@lists.fedoraproject.org


[Bug 1283540] perl-Mojolicious-Plugin-CHI-0.12 is available

2015-11-29 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1283540

Emmanuel Seyman  changed:

   What|Removed |Added

 Status|NEW |CLOSED
   Fixed In Version||perl-Mojolicious-Plugin-CHI
   ||-0.12-1.fc24
 Resolution|--- |RAWHIDE
Last Closed||2015-11-29 10:25:20



-- 
You are receiving this mail because:
You are on the CC list for the bug.
--
Fedora Extras Perl SIG
http://www.fedoraproject.org/wiki/Extras/SIGs/Perl
perl-devel mailing list
perl-devel@lists.fedoraproject.org
http://lists.fedoraproject.org/admin/lists/perl-devel@lists.fedoraproject.org


[Bug 1285570] perl-Hijk-0.26 is available

2015-11-29 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1285570

Emmanuel Seyman  changed:

   What|Removed |Added

 Status|NEW |CLOSED
   Fixed In Version||perl-Hijk-0.26-1.fc24
 Resolution|--- |RAWHIDE
Last Closed||2015-11-29 10:24:53



-- 
You are receiving this mail because:
You are on the CC list for the bug.
--
Fedora Extras Perl SIG
http://www.fedoraproject.org/wiki/Extras/SIGs/Perl
perl-devel mailing list
perl-devel@lists.fedoraproject.org
http://lists.fedoraproject.org/admin/lists/perl-devel@lists.fedoraproject.org


[Bug 1283442] perl-Mojolicious-6.33 is available

2015-11-29 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1283442

Emmanuel Seyman  changed:

   What|Removed |Added

 Status|NEW |CLOSED
   Fixed In Version||perl-Mojolicious-6.33-1.fc2
   ||4
 Resolution|--- |RAWHIDE
Last Closed||2015-11-29 10:25:45



-- 
You are receiving this mail because:
You are on the CC list for the bug.
--
Fedora Extras Perl SIG
http://www.fedoraproject.org/wiki/Extras/SIGs/Perl
perl-devel mailing list
perl-devel@lists.fedoraproject.org
http://lists.fedoraproject.org/admin/lists/perl-devel@lists.fedoraproject.org


[Bug 1277233] Please branch and build for EPEL 7

2015-11-29 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1277233

Emmanuel Seyman  changed:

   What|Removed |Added

   Assignee|emman...@seyman.fr  |limburg...@gmail.com



--- Comment #5 from Emmanuel Seyman  ---
Jon, I'm assigning this bug to you since you're doing the heavy lifting.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
--
Fedora Extras Perl SIG
http://www.fedoraproject.org/wiki/Extras/SIGs/Perl
perl-devel mailing list
perl-devel@lists.fedoraproject.org
http://lists.fedoraproject.org/admin/lists/perl-devel@lists.fedoraproject.org


Fedora Rawhide 20151129 compose check report

2015-11-29 Thread Fedora compose checker
Missing expected images:

Cloud disk raw i386
Cloud disk raw x86_64
Cloud_atomic disk raw x86_64
Workstation live i386
Workstation live x86_64

No images in this compose but not Rawhide 20151128

No images in Rawhide 20151128 but not this.

Failed openQA tests: 48 of 49

ID: 9223Test: i386 kde_live default_install
ID: 9222Test: x86_64 kde_live default_install
ID: 9221Test: i386 generic_boot default_install
ID: 9220Test: x86_64 generic_boot default_install@uefi
ID: 9219Test: x86_64 generic_boot default_install
ID: 9218Test: i386 universal upgrade_desktop_32bit
ID: 9217Test: i386 universal server_lvmthin
ID: 9216Test: i386 universal server_ext3
ID: 9215Test: i386 universal server_btrfs
ID: 9214Test: i386 universal server_software_raid
ID: 9213Test: i386 universal server_simple_encrypted
ID: 9212Test: i386 universal server_repository_http_graphical
ID: 9211Test: i386 universal server_scsi_updates_img
ID: 9210Test: i386 universal package_set_minimal
ID: 9209Test: x86_64 universal server_no_swap@uefi
ID: 9208Test: x86_64 universal server_lvmthin@uefi
ID: 9207Test: x86_64 universal server_ext3@uefi
ID: 9206Test: x86_64 universal server_btrfs@uefi
ID: 9205Test: x86_64 universal server_software_raid@uefi
ID: 9204Test: x86_64 universal server_multi_empty@uefi
ID: 9203Test: x86_64 universal server_simple_free_space@uefi
ID: 9202Test: x86_64 universal server_simple_encrypted@uefi
ID: 9201Test: x86_64 universal server_delete_partial@uefi
ID: 9200Test: x86_64 universal server_delete_pata@uefi
ID: 9199Test: x86_64 universal server_sata_multi@uefi
ID: 9198Test: x86_64 universal european_language_install
ID: 9197Test: x86_64 universal server_shrink_ntfs
ID: 9196Test: x86_64 universal server_shrink_ext4
ID: 9195Test: x86_64 universal server_updates_img_local
ID: 9194Test: x86_64 universal upgrade_desktop_64bit
ID: 9192Test: x86_64 universal server_kickstart_hdd
ID: 9191Test: x86_64 universal server_no_swap
ID: 9190Test: x86_64 universal server_lvmthin
ID: 9189Test: x86_64 universal server_ext3
ID: 9188Test: x86_64 universal server_btrfs
ID: 9187Test: x86_64 universal server_software_raid
ID: 9186Test: x86_64 universal server_multi_empty
ID: 9185Test: x86_64 universal server_simple_free_space
ID: 9184Test: x86_64 universal server_simple_encrypted
ID: 9183Test: x86_64 universal server_delete_partial
ID: 9182Test: x86_64 universal server_repository_http_variation
ID: 9181Test: x86_64 universal server_repository_http_graphical
ID: 9180Test: x86_64 universal server_mirrorlist_graphical
ID: 9179Test: x86_64 universal server_delete_pata
ID: 9178Test: x86_64 universal server_kickstart_user_creation
ID: 9177Test: x86_64 universal server_scsi_updates_img
ID: 9176Test: x86_64 universal server_sata_multi
ID: 9175Test: x86_64 universal package_set_minimal

Passed openQA tests: 1 of 49
-- 
Mail generated by check-compose:
https://git.fedorahosted.org/cgit/fedora-qa.git/tree/check-compose
--
devel mailing list
devel@lists.fedoraproject.org
http://lists.fedoraproject.org/admin/lists/devel@lists.fedoraproject.org


pghmcfc pushed to perl-Crypt-Random (master). "Fix broken previous fix for CPAN RT#99880, RHBZ#1158379"

2015-11-29 Thread notifications
>From 6aa8938158eb8379b86f869b75f5df77ab3d274f Mon Sep 17 00:00:00 2001
From: Paul Howarth 
Date: Sun, 29 Nov 2015 16:02:02 +
Subject: Fix broken previous fix for CPAN RT#99880, RHBZ#1158379

---
 Crypt-Random-1.25-test-fix.patch | 17 -
 perl-Crypt-Random.spec   |  8 +++-
 2 files changed, 19 insertions(+), 6 deletions(-)

diff --git a/Crypt-Random-1.25-test-fix.patch b/Crypt-Random-1.25-test-fix.patch
index 7c2a822..88cf7d3 100644
--- a/Crypt-Random-1.25-test-fix.patch
+++ b/Crypt-Random-1.25-test-fix.patch
@@ -1,13 +1,20 @@
 --- t/generator.t
 +++ t/generator.t
-@@ -23,7 +23,10 @@
+@@ -10,7 +10,7 @@
+ 
+ use Test;
+ use Crypt::Random::Generator;
+-BEGIN { plan tests => 18 };
++BEGIN { plan tests => 18, todo => [3,9,15] };
+ 
+ tests( new Crypt::Random::Generator Strength => 1 );
+ tests( new Crypt::Random::Generator Strength => 0 );
+@@ -23,7 +23,7 @@ sub tests {
  my $y = $gen->integer (Size => 10);
  ok($x < 1025, 1);
  ok($y < 1025, 1);
-+TODO: {
-+local $TODO = "There's about a 0.1% chance that this test will fail 
normally";
- ok($x != $y, 1);
-+}
+-ok($x != $y, 1);
++ok($x != $y, 1); # About 0.1% chance of this test failing, so it's TODO
  ok($gen->integer (Upper => 500) < 501, 1);
  ok($gen->integer (Size => 128));
  ok(length($gen->string (Length => 30)), 30);
diff --git a/perl-Crypt-Random.spec b/perl-Crypt-Random.spec
index 96ffae0..6913ed2 100644
--- a/perl-Crypt-Random.spec
+++ b/perl-Crypt-Random.spec
@@ -1,7 +1,7 @@
 Summary:   Cryptographically Secure, True Random Number Generator
 Name:  perl-Crypt-Random
 Version:   1.25
-Release:   23%{?dist}
+Release:   24%{?dist}
 License:   GPL+ or Artistic
 Group: Development/Libraries
 Url:   http://search.cpan.org/dist/Crypt-Random/
@@ -10,6 +10,9 @@ Patch0:   Crypt-Random-1.25-test-fix.patch
 BuildRoot: %{_tmppath}/%{name}-%{version}-%{release}-root-%(id -nu)
 BuildArch: noarch
 # Module Build
+BuildRequires: coreutils
+BuildRequires: findutils
+BuildRequires: make
 BuildRequires: perl
 BuildRequires: perl(ExtUtils::MakeMaker)
 # Module Runtime
@@ -69,6 +72,9 @@ rm -rf %{buildroot}
 %{_mandir}/man3/Crypt::Random.3*
 
 %changelog
+* Sun Nov 29 2015 Paul Howarth  - 1.25-24
+- Fix broken previous fix for CPAN RT#99880, RHBZ#1158379
+
 * Thu Jun 18 2015 Fedora Release Engineering  
- 1.25-23
 - Rebuilt for https://fedoraproject.org/wiki/Fedora_23_Mass_Rebuild
 
-- 
cgit v0.11.2



http://pkgs.fedoraproject.org/cgit/perl-Crypt-Random.git/commit/?h=master=6aa8938158eb8379b86f869b75f5df77ab3d274f
--
Fedora Extras Perl SIG
http://www.fedoraproject.org/wiki/Extras/SIGs/Perl
perl-devel mailing list
perl-devel@lists.fedoraproject.org
http://lists.fedoraproject.org/admin/lists/perl-devel@lists.fedoraproject.org


pghmcfc pushed to perl-Crypt-Random (perl-Crypt-Random-1.25-24.fc24). "Fix broken previous fix for CPAN RT#99880, RHBZ#1158379"

2015-11-29 Thread notifications
>From 6aa8938158eb8379b86f869b75f5df77ab3d274f Mon Sep 17 00:00:00 2001
From: Paul Howarth 
Date: Sun, 29 Nov 2015 16:02:02 +
Subject: Fix broken previous fix for CPAN RT#99880, RHBZ#1158379

---
 Crypt-Random-1.25-test-fix.patch | 17 -
 perl-Crypt-Random.spec   |  8 +++-
 2 files changed, 19 insertions(+), 6 deletions(-)

diff --git a/Crypt-Random-1.25-test-fix.patch b/Crypt-Random-1.25-test-fix.patch
index 7c2a822..88cf7d3 100644
--- a/Crypt-Random-1.25-test-fix.patch
+++ b/Crypt-Random-1.25-test-fix.patch
@@ -1,13 +1,20 @@
 --- t/generator.t
 +++ t/generator.t
-@@ -23,7 +23,10 @@
+@@ -10,7 +10,7 @@
+ 
+ use Test;
+ use Crypt::Random::Generator;
+-BEGIN { plan tests => 18 };
++BEGIN { plan tests => 18, todo => [3,9,15] };
+ 
+ tests( new Crypt::Random::Generator Strength => 1 );
+ tests( new Crypt::Random::Generator Strength => 0 );
+@@ -23,7 +23,7 @@ sub tests {
  my $y = $gen->integer (Size => 10);
  ok($x < 1025, 1);
  ok($y < 1025, 1);
-+TODO: {
-+local $TODO = "There's about a 0.1% chance that this test will fail 
normally";
- ok($x != $y, 1);
-+}
+-ok($x != $y, 1);
++ok($x != $y, 1); # About 0.1% chance of this test failing, so it's TODO
  ok($gen->integer (Upper => 500) < 501, 1);
  ok($gen->integer (Size => 128));
  ok(length($gen->string (Length => 30)), 30);
diff --git a/perl-Crypt-Random.spec b/perl-Crypt-Random.spec
index 96ffae0..6913ed2 100644
--- a/perl-Crypt-Random.spec
+++ b/perl-Crypt-Random.spec
@@ -1,7 +1,7 @@
 Summary:   Cryptographically Secure, True Random Number Generator
 Name:  perl-Crypt-Random
 Version:   1.25
-Release:   23%{?dist}
+Release:   24%{?dist}
 License:   GPL+ or Artistic
 Group: Development/Libraries
 Url:   http://search.cpan.org/dist/Crypt-Random/
@@ -10,6 +10,9 @@ Patch0:   Crypt-Random-1.25-test-fix.patch
 BuildRoot: %{_tmppath}/%{name}-%{version}-%{release}-root-%(id -nu)
 BuildArch: noarch
 # Module Build
+BuildRequires: coreutils
+BuildRequires: findutils
+BuildRequires: make
 BuildRequires: perl
 BuildRequires: perl(ExtUtils::MakeMaker)
 # Module Runtime
@@ -69,6 +72,9 @@ rm -rf %{buildroot}
 %{_mandir}/man3/Crypt::Random.3*
 
 %changelog
+* Sun Nov 29 2015 Paul Howarth  - 1.25-24
+- Fix broken previous fix for CPAN RT#99880, RHBZ#1158379
+
 * Thu Jun 18 2015 Fedora Release Engineering  
- 1.25-23
 - Rebuilt for https://fedoraproject.org/wiki/Fedora_23_Mass_Rebuild
 
-- 
cgit v0.11.2



http://pkgs.fedoraproject.org/cgit/perl-Crypt-Random.git/commit/?h=perl-Crypt-Random-1.25-24.fc24=6aa8938158eb8379b86f869b75f5df77ab3d274f
--
Fedora Extras Perl SIG
http://www.fedoraproject.org/wiki/Extras/SIGs/Perl
perl-devel mailing list
perl-devel@lists.fedoraproject.org
http://lists.fedoraproject.org/admin/lists/perl-devel@lists.fedoraproject.org


Re: Dealing with the "my packages" problem

2015-11-29 Thread Neal Gompa
On Sun, Nov 29, 2015 at 7:00 PM, Nico Kadel-Garcia  wrote:
>
> On Wed, Nov 18, 2015 at 4:49 PM, Adam Williamson
>  wrote:
>
> > Just as a general note on this thread: if you're a packager and you
> > have a genuine reason why people should be careful about touching your
> > package, or follow some specific process when doing so, or there's
> > something that people might think they should change but they
> > shouldn't, there is already a pretty effective way of dealing with
> > this:
> >
> > ** PUT A COMMENT IN THE SPEC FILE **
> >
> > this is extremely easy to do, and extremely difficult for anyone who
> > touches it to claim they didn't see.
>
> I know this thread is a bit old, but thought I'd add a note when I
> checked my backlog and saw it.
>
> A comment in the code accompanying the change itself can also be
> invaluable, especially for changes that should propagate upstream.I
> cannot count the number of patches that have *nothing* in the patch
> content, only in the comments preceding the patch itself. I'm
> personally exhausted with people who refuse to document, whose API has
> to be deduced from the code, and especially whose code could fit
> *mutliple* workflows. And worse, when the API they think they're using
> isn't the one they've actually written, and they get upset when you
> don't code to this non-published API.
>
> That's cost me a job I rather liked.
>
> > Package builds in a community distro are really just like F/OSS code:
> > you should assume other people are going to be looking at it and poking
> > it and trying to do stuff with it, and any time it's doing something
> > that isn't extremely obvious or diverges from the general conventions,
> > it makes sense to comment it.
>
> Yes, lordie, please, "the code is the documentation" is an attitude
> I've encountered far, far too often, and I consider it the sign of a
> dangerous programmer. It's afraid it's a job security thing: by
> refusing to document you protect your turf.
>

I'd like to add to this by pleading to *not* strip the comment section
out of a patch/diff when you pull them from upstream/git/pull
requests/etc. git format-patch is your friend in the Gitverse, and
similar tools exist across all major SCM systems.

Maybe it's because some people don't know about -p1 or something, but
I've seen people strip and partially rewrite patches in the past so
that they would apply. I was also guilty of this before I figured out
how to use %patch and later learned of %autosetup/%autopatch, so I
don't know if it's because people have issues with that or something
else.




-- 
真実はいつも一つ!/ Always, there's only one truth!
--
devel mailing list
devel@lists.fedoraproject.org
http://lists.fedoraproject.org/admin/lists/devel@lists.fedoraproject.org

[Bug 1282917] Review Request: perl-Test-mysqld - Mysqld runner for tests

2015-11-29 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1282917



--- Comment #17 from Fedora Update System  ---
perl-Test-mysqld-0.17-3.fc23 has been pushed to the Fedora 23 stable
repository. If problems still persist, please make note of it in this bug
report.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
--
Fedora Extras Perl SIG
http://www.fedoraproject.org/wiki/Extras/SIGs/Perl
perl-devel mailing list
perl-devel@lists.fedoraproject.org
http://lists.fedoraproject.org/admin/lists/perl-devel@lists.fedoraproject.org


[Bug 1282917] Review Request: perl-Test-mysqld - Mysqld runner for tests

2015-11-29 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1282917

Fedora Update System  changed:

   What|Removed |Added

 Status|ON_QA   |CLOSED
 Resolution|--- |ERRATA
Last Closed||2015-11-29 21:19:32



-- 
You are receiving this mail because:
You are on the CC list for the bug.
--
Fedora Extras Perl SIG
http://www.fedoraproject.org/wiki/Extras/SIGs/Perl
perl-devel mailing list
perl-devel@lists.fedoraproject.org
http://lists.fedoraproject.org/admin/lists/perl-devel@lists.fedoraproject.org


[Bug 1282917] Review Request: perl-Test-mysqld - Mysqld runner for tests

2015-11-29 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1282917



--- Comment #18 from Fedora Update System  ---
perl-Test-mysqld-0.17-3.fc22 has been pushed to the Fedora 22 stable
repository. If problems still persist, please make note of it in this bug
report.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
--
Fedora Extras Perl SIG
http://www.fedoraproject.org/wiki/Extras/SIGs/Perl
perl-devel mailing list
perl-devel@lists.fedoraproject.org
http://lists.fedoraproject.org/admin/lists/perl-devel@lists.fedoraproject.org


[Bug 1094440] CVE-2014-3230 perl-libwww-perl: incorrect handling of SSL certificate verification

2015-11-29 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1094440

Ján Rusnačko  changed:

   What|Removed |Added

 CC||jrusn...@redhat.com
 Whiteboard|impact=moderate,public=2014 |impact=moderate,public=2014
   |0501,reported=20140501,sour |0501,reported=20140501,sour
   |ce=debian,cvss2=5.8/AV:N/AC |ce=debian,cvss2=5.8/AV:N/AC
   |:M/Au:N/C:P/I:P/A:N,rhel-5/ |:M/Au:N/C:P/I:P/A:N,rhel-5/
   |perl-libwww-perl=notaffecte |perl-libwww-perl=notaffecte
   |d,rhel-6/perl-libwww-perl=n |d,rhel-6/perl-libwww-perl=n
   |otaffected,rhel-7/perl-libw |otaffected,rhel-7/perl-libw
   |ww-perl=wontfix,fedora-all/ |ww-perl=wontfix,fedora-all/
   |perl-libwww-perl=affected   |perl-libwww-perl=affected,c
   ||we=CWE-295



-- 
You are receiving this mail because:
You are on the CC list for the bug.
--
Fedora Extras Perl SIG
http://www.fedoraproject.org/wiki/Extras/SIGs/Perl
perl-devel mailing list
perl-devel@lists.fedoraproject.org
http://lists.fedoraproject.org/admin/lists/perl-devel@lists.fedoraproject.org

Re: Dealing with the "my packages" problem

2015-11-29 Thread Nico Kadel-Garcia
On Wed, Nov 18, 2015 at 4:49 PM, Adam Williamson
 wrote:

> Just as a general note on this thread: if you're a packager and you
> have a genuine reason why people should be careful about touching your
> package, or follow some specific process when doing so, or there's
> something that people might think they should change but they
> shouldn't, there is already a pretty effective way of dealing with
> this:
>
> ** PUT A COMMENT IN THE SPEC FILE **
>
> this is extremely easy to do, and extremely difficult for anyone who
> touches it to claim they didn't see.

I know this thread is a bit old, but thought I'd add a note when I
checked my backlog and saw it.

A comment in the code accompanying the change itself can also be
invaluable, especially for changes that should propagate upstream.I
cannot count the number of patches that have *nothing* in the patch
content, only in the comments preceding the patch itself. I'm
personally exhausted with people who refuse to document, whose API has
to be deduced from the code, and especially whose code could fit
*mutliple* workflows. And worse, when the API they think they're using
isn't the one they've actually written, and they get upset when you
don't code to this non-published API.

That's cost me a job I rather liked.

> Package builds in a community distro are really just like F/OSS code:
> you should assume other people are going to be looking at it and poking
> it and trying to do stuff with it, and any time it's doing something
> that isn't extremely obvious or diverges from the general conventions,
> it makes sense to comment it.

Yes, lordie, please, "the code is the documentation" is an attitude
I've encountered far, far too often, and I consider it the sign of a
dangerous programmer. It's afraid it's a job security thing: by
refusing to document you protect your turf.
--
devel mailing list
devel@lists.fedoraproject.org
http://lists.fedoraproject.org/admin/lists/devel@lists.fedoraproject.org


Re: Koji build fails, scratch-build works

2015-11-29 Thread Michael Schwendt
On Sun, 29 Nov 2015 19:10:25 +0100, Mattia Verga wrote:

> Solved by changing .spec file name as set in PackageDB.
> 
> The developer himself uses both kpmcore and KPMcore in the project, so I 
> think there's no problem in adopting KPMcore as name in Fedora.
> 
> Sorry for the noise.

Both
  https://fedoraproject.org/wiki/Packaging:Guidelines#Spec_File_Naming
and
  https://fedoraproject.org/wiki/Packaging:NamingGuidelines#General_Naming
apply.

And it can be quite tiresome, if such issues come up during review.
--
devel mailing list
devel@lists.fedoraproject.org
http://lists.fedoraproject.org/admin/lists/devel@lists.fedoraproject.org


Re: Koji build fails, scratch-build works

2015-11-29 Thread Michael Schwendt
On Sun, 29 Nov 2015 18:50:22 +, Tom Hughes wrote:

> It always used to be the case that the request for a package to be added 
> to pkgdb was rejected if the title didn't match,

True. For a long time, reviewers have adjusted the bug title during review
because of that.

> so has a bug been 
> introduced there by the new pkgdb process?

That's likely. Considering that a completely unreviewed package (lacking
fedora-review+ flag even) has been approved recently, I guess there are
some problems in the process.

All that's IMHO.
--
devel mailing list
devel@lists.fedoraproject.org
http://lists.fedoraproject.org/admin/lists/devel@lists.fedoraproject.org