[Bug 226285] Merge Review: perl-XML-Grove
by Red Hat Bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=226285
--- Comment #31 from Patrice Dumas <pertusus(a)free.fr> 2008-10-15 10:09:00 EDT ---
(In reply to comment #30)
> (In reply to comment #28)
> > I am missing something, isn't the file in the lookaside cache since a long time
> > now?
>
> No, it isn't; the file in cvs has yesterday's time stamp:
I forgot about the .tar.gz vs .tar.bz2.
> I think that the bug in upload.cgi which caused that, should be reported.
I guess that the best location would be the trac of the infrastructure team.
--
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.
15 years, 6 months
[Bug 226285] Merge Review: perl-XML-Grove
by Red Hat Bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=226285
--- Comment #30 from Stepan Kasal <skasal(a)redhat.com> 2008-10-15 10:06:07 EDT ---
(In reply to comment #28)
> I am missing something, isn't the file in the lookaside cache since a long time
> now?
No, it isn't; the file in cvs has yesterday's time stamp:
> -rw-rw-r-- 1 dumas dumas 27336 oct. 14 12:13 XML-Grove-0.46alpha.tar.gz-cvs
and I imported that file yesterday. And I did the import yesterday, using the
originial tar.gz, dowloaded from upstream, with the right 1999 time stamp.
But the upload command (upload.cgi, called from "make new-source") screwed the
time stamp. I do not know why.
I agree with you that there is no known way to fix the time stamp now.
I think that the bug in upload.cgi which caused that, should be reported.
--
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.
15 years, 6 months
[Bug 226285] Merge Review: perl-XML-Grove
by Red Hat Bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=226285
--- Comment #29 from Stepan Kasal <skasal(a)redhat.com> 2008-10-15 09:59:54 EDT ---
(In reply to comment #24)
> There are spurious provides coming from examples:
>
> Provides: perl(MyHTML) perl(MyVisitor)
indeed. So it seems the messages were almost right:
> perl-XML-Grove.noarch: W: doc-file-dependency
> /usr/share/doc/perl-XML-Grove-0.46alpha/examples/my-html.pl
> perl(XML::Parser::PerlSAX)
> perl-XML-Grove.noarch: W: doc-file-dependency
> /usr/share/doc/perl-XML-Grove-0.46alpha/examples/visitor.pl
> perl(XML::Parser::PerlSAX)
Had it mentioned perl(MyHTML) and perl(MyVisitor), respectively, the messages
would have named the problem.
But the "rpmlint -i" hint says that it helps to clear the exec bit of the
files.
This is not true, the bit is cleared, yet the dependency generator brings in
the wrong provides.
Should I fight against the dependency generator somehow? gzipping the examples,
rot13-encoding, renaming them, clearing the #! line or whatever... ???
Or is it enough to file a bug aginst the buggy dependency generator?
To sum up, I still apply for the approval of the package in its current state
in the cvs.
--
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.
15 years, 6 months
[Bug 226285] Merge Review: perl-XML-Grove
by Red Hat Bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=226285
--- Comment #28 from Patrice Dumas <pertusus(a)free.fr> 2008-10-15 09:49:28 EDT ---
I am missing something, isn't the file in the lookaside cache since a long time
now? Subsequente make upload won't change the timestamp. In fact I guess that
it was automatically imported, or something like that, during the marge from
the old Red Haat internal buildsystem.
Anyway it isn't important, just fix the Provides and we're done.
--
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.
15 years, 6 months
[Bug 226285] Merge Review: perl-XML-Grove
by Red Hat Bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=226285
--- Comment #27 from Stepan Kasal <skasal(a)redhat.com> 2008-10-15 09:26:09 EDT ---
(In reply to comment #26)
I said:
> I think that this upload failed because I had not the ssh key in my ssh
> keyring at the moment.
This does not seem likely, the ssh key is used later, for "cvs update".
> But in any case, it looks like a bug that "make upload" left
> things in an inconsistent state.
Indeed, it looks like a bug in update.cgi; don't you, by any chance, know
against what component should the bug be filed?
--
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.
15 years, 6 months
[Bug 226285] Merge Review: perl-XML-Grove
by Red Hat Bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=226285
--- Comment #26 from Stepan Kasal <skasal(a)redhat.com> 2008-10-15 09:13:04 EDT ---
(In reply to comment #24)
> ls -l *gz*
> -rw-rw-r-- 1 dumas dumas 27336 sept. 10 1999 XML-Grove-0.46alpha.tar.gz
> -rw-rw-r-- 1 dumas dumas 27336 oct. 14 12:13 XML-Grove-0.46alpha.tar.gz-cvs
> tarball timestamp was not kept. It is too late now, but please do it
> for the next time.
Weird. My own copy of the file had the right time stamp when I issued
make new-source FILES=XML-Grove-0.46alpha.tar.gz
But it seems the upload process has crippled the time stamp.
2008-10-14 12:13:51 is the access time of the file on my disk, but I do not
know if it is the time of download from upstream or the time of upload to
fedora cvs.
I think that this upload failed because I had not the ssh key in my ssh keyring
at the moment. But in any case, it looks like a bug that "make upload" left
things in an inconsistent state.
--
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.
15 years, 6 months
rpms/perl-Padre/F-10 Padre-removevsORLite.patch, NONE, 1.1 perl-Padre.spec, NONE, 1.1 .cvsignore, 1.1, 1.2 sources, 1.1, 1.2
by Marcela Mašláňová
Author: mmaslano
Update of /cvs/pkgs/rpms/perl-Padre/F-10
In directory cvs1.fedora.phx.redhat.com:/tmp/cvs-serv19337
Modified Files:
.cvsignore sources
Added Files:
Padre-removevsORLite.patch perl-Padre.spec
Log Message:
* Fri Oct 10 2008 Marcela Mašláňová <mmaslano(a)redhat.com> 0.10-4
- move templates into _datadir
Padre-removevsORLite.patch:
--- NEW FILE Padre-removevsORLite.patch ---
diff -up Padre-0.10/lib/Padre/DB.pm.old Padre-0.10/lib/Padre/DB.pm
--- Padre-0.10/lib/Padre/DB.pm.old 2008-09-22 07:50:46.000000000 +0200
+++ Padre-0.10/lib/Padre/DB.pm 2008-10-03 09:45:23.000000000 +0200
@@ -4,11 +4,10 @@ package Padre::DB;
use strict;
use Padre;
-use ORLite 0.13 {
+use ORLite {
file => Padre->ide->config_db,
tables => 0,
};
-our $VERSION = '0.10';
1;
--- NEW FILE perl-Padre.spec ---
Name: perl-Padre
Version: 0.10
Release: 4%{?dist}
Summary: Perl Application Development and Refactoring Environment
License: GPL+ or Artistic
Group: Development/Libraries
URL: http://search.cpan.org/dist/Padre/
Source0: http://www.cpan.org/authors/id/S/SZ/SZABGAB/Padre-%{version}.tar.gz
Source1: Padre.desktop
Patch0: Padre-removevsORLite.patch
BuildRoot: %{_tmppath}/%{name}-%{version}-%{release}-root-%(%{__id_u} -n)
BuildArch: noarch
BuildRequires: perl >= 0:5.008
BuildRequires: perl(Class::Accessor)
BuildRequires: perl(DBD::SQLite)
BuildRequires: perl(File::Copy::Recursive)
BuildRequires: perl(Module::Build)
BuildRequires: perl(PAR) >= 0.970
BuildRequires: perl(Pod::Simple)
BuildRequires: perl(Probe::Perl) >= 0.01
BuildRequires: perl(Test::Exception)
BuildRequires: perl(Test::More) >= 0.47
BuildRequires: perl(Test::NoWarnings)
BuildRequires: perl(Wx::Perl::Dialog)
BuildRequires: desktop-file-utils
Requires: perl(Class::Accessor)
Requires: perl(DBD::SQLite)
Requires: perl(File::Copy::Recursive)
Requires: perl(PAR) >= 0.970
Requires: perl(Pod::Simple)
Requires: perl(PPI) >= 1.203
Requires: perl(Probe::Perl) >= 0.01
Requires: perl(Wx::Perl::Dialog)
Requires: perl(:MODULE_COMPAT_%(eval "`%{__perl} -V:version`"; echo $version))
%description
Padre is a text editor aimed to be an IDE for Perl.
The application maintains its configuration information
in a directory called .padre.
%prep
%setup -q -n Padre-%{version}
%patch0 -p1 -b .patch
%build
%{__perl} Build.PL installdirs=vendor
./Build
%install
rm -rf $RPM_BUILD_ROOT
./Build install destdir=$RPM_BUILD_ROOT create_packlist=0
find $RPM_BUILD_ROOT -depth -type d -exec rmdir {} 2>/dev/null \;
mv $RPM_BUILD_ROOT/%{perl_vendorlib}/auto/share/dist/Padre $RPM_BUILD_ROOT/%{_datadir}/padre
%{_fixperms} $RPM_BUILD_ROOT/*
chmod 644 $RPM_BUILD_ROOT/%{perl_vendorlib}/Padre/Wx/Ack.pm
# there's no desktop file in source, let's create one
desktop-file-install --vendor="fedora" \
--dir=$RPM_BUILD_ROOT/%{_datadir}/applications \
%{SOURCE1}
%check
# We don't test GUI tests.
#./Build test
%clean
rm -rf $RPM_BUILD_ROOT
%files
%defattr(-,root,root,-)
%doc Changes README
/usr/bin/padre
%{perl_vendorlib}/*
%{_datadir}/padre
%{_datadir}/applications/fedora-Padre.desktop
%{_mandir}/man3/*
%changelog
* Fri Oct 10 2008 Marcela Mašláňová <mmaslano(a)redhat.com> 0.10-4
- move templates into _datadir
* Fri Oct 10 2008 Marcela Mašláňová <mmaslano(a)redhat.com> 0.10-3
- add desktop file
* Fri Oct 10 2008 Marcela Mašláňová <mmaslano(a)redhat.com> 0.10-2
- fix review
- example and test files should have permission 644
- desktop file is missing in upstream version
* Fri Oct 03 2008 Marcela Mašláňová <mmaslano(a)redhat.com> 0.10-1
- Specfile autogenerated by cpanspec 1.77.
Index: .cvsignore
===================================================================
RCS file: /cvs/pkgs/rpms/perl-Padre/F-10/.cvsignore,v
retrieving revision 1.1
retrieving revision 1.2
diff -u -r1.1 -r1.2
--- .cvsignore 15 Oct 2008 09:59:35 -0000 1.1
+++ .cvsignore 15 Oct 2008 10:51:49 -0000 1.2
@@ -0,0 +1,2 @@
+Padre-0.10.tar.gz
+Padre.desktop
Index: sources
===================================================================
RCS file: /cvs/pkgs/rpms/perl-Padre/F-10/sources,v
retrieving revision 1.1
retrieving revision 1.2
diff -u -r1.1 -r1.2
--- sources 15 Oct 2008 09:59:35 -0000 1.1
+++ sources 15 Oct 2008 10:51:49 -0000 1.2
@@ -0,0 +1,2 @@
+d036a33e475a962415e4ef9837c7a74e Padre-0.10.tar.gz
+e9457439b1f0c929ab3141197294e061 Padre.desktop
15 years, 6 months
rpms/perl-Padre/devel Padre-removevsORLite.patch, NONE, 1.1 .cvsignore, 1.2, 1.3 sources, 1.2, 1.3
by Marcela Mašláňová
Author: mmaslano
Update of /cvs/pkgs/rpms/perl-Padre/devel
In directory cvs1.fedora.phx.redhat.com:/tmp/cvs-serv17008
Modified Files:
.cvsignore sources
Added Files:
Padre-removevsORLite.patch
Log Message:
* Fri Oct 10 2008 Marcela Mašláňová <mmaslano(a)redhat.com> 0.10-4
- move templates into _datadir
Padre-removevsORLite.patch:
--- NEW FILE Padre-removevsORLite.patch ---
diff -up Padre-0.10/lib/Padre/DB.pm.old Padre-0.10/lib/Padre/DB.pm
--- Padre-0.10/lib/Padre/DB.pm.old 2008-09-22 07:50:46.000000000 +0200
+++ Padre-0.10/lib/Padre/DB.pm 2008-10-03 09:45:23.000000000 +0200
@@ -4,11 +4,10 @@ package Padre::DB;
use strict;
use Padre;
-use ORLite 0.13 {
+use ORLite {
file => Padre->ide->config_db,
tables => 0,
};
-our $VERSION = '0.10';
1;
Index: .cvsignore
===================================================================
RCS file: /cvs/pkgs/rpms/perl-Padre/devel/.cvsignore,v
retrieving revision 1.2
retrieving revision 1.3
diff -u -r1.2 -r1.3
--- .cvsignore 15 Oct 2008 10:21:05 -0000 1.2
+++ .cvsignore 15 Oct 2008 10:28:14 -0000 1.3
@@ -1 +1,2 @@
Padre-0.10.tar.gz
+Padre.desktop
Index: sources
===================================================================
RCS file: /cvs/pkgs/rpms/perl-Padre/devel/sources,v
retrieving revision 1.2
retrieving revision 1.3
diff -u -r1.2 -r1.3
--- sources 15 Oct 2008 10:21:05 -0000 1.2
+++ sources 15 Oct 2008 10:28:14 -0000 1.3
@@ -1 +1,2 @@
d036a33e475a962415e4ef9837c7a74e Padre-0.10.tar.gz
+e9457439b1f0c929ab3141197294e061 Padre.desktop
15 years, 6 months
rpms/perl-Padre/devel perl-Padre.spec, NONE, 1.1 .cvsignore, 1.1, 1.2 sources, 1.1, 1.2
by Marcela Mašláňová
Author: mmaslano
Update of /cvs/pkgs/rpms/perl-Padre/devel
In directory cvs1.fedora.phx.redhat.com:/tmp/cvs-serv15854
Modified Files:
.cvsignore sources
Added Files:
perl-Padre.spec
Log Message:
* Fri Oct 10 2008 Marcela Mašláňová <mmaslano(a)redhat.com> 0.10-4
- move templates into _datadir
--- NEW FILE perl-Padre.spec ---
Name: perl-Padre
Version: 0.10
Release: 4%{?dist}
Summary: Perl Application Development and Refactoring Environment
License: GPL+ or Artistic
Group: Development/Libraries
URL: http://search.cpan.org/dist/Padre/
Source0: http://www.cpan.org/authors/id/S/SZ/SZABGAB/Padre-%{version}.tar.gz
Source1: Padre.desktop
Patch0: Padre-removevsORLite.patch
BuildRoot: %{_tmppath}/%{name}-%{version}-%{release}-root-%(%{__id_u} -n)
BuildArch: noarch
BuildRequires: perl >= 0:5.008
BuildRequires: perl(Class::Accessor)
BuildRequires: perl(DBD::SQLite)
BuildRequires: perl(File::Copy::Recursive)
BuildRequires: perl(Module::Build)
BuildRequires: perl(PAR) >= 0.970
BuildRequires: perl(Pod::Simple)
BuildRequires: perl(Probe::Perl) >= 0.01
BuildRequires: perl(Test::Exception)
BuildRequires: perl(Test::More) >= 0.47
BuildRequires: perl(Test::NoWarnings)
BuildRequires: perl(Wx::Perl::Dialog)
BuildRequires: desktop-file-utils
Requires: perl(Class::Accessor)
Requires: perl(DBD::SQLite)
Requires: perl(File::Copy::Recursive)
Requires: perl(PAR) >= 0.970
Requires: perl(Pod::Simple)
Requires: perl(PPI) >= 1.203
Requires: perl(Probe::Perl) >= 0.01
Requires: perl(Wx::Perl::Dialog)
Requires: perl(:MODULE_COMPAT_%(eval "`%{__perl} -V:version`"; echo $version))
%description
Padre is a text editor aimed to be an IDE for Perl.
The application maintains its configuration information
in a directory called .padre.
%prep
%setup -q -n Padre-%{version}
%patch0 -p1 -b .patch
%build
%{__perl} Build.PL installdirs=vendor
./Build
%install
rm -rf $RPM_BUILD_ROOT
./Build install destdir=$RPM_BUILD_ROOT create_packlist=0
find $RPM_BUILD_ROOT -depth -type d -exec rmdir {} 2>/dev/null \;
mv $RPM_BUILD_ROOT/%{perl_vendorlib}/auto/share/dist/Padre $RPM_BUILD_ROOT/%{_datadir}/padre
%{_fixperms} $RPM_BUILD_ROOT/*
chmod 644 $RPM_BUILD_ROOT/%{perl_vendorlib}/Padre/Wx/Ack.pm
# there's no desktop file in source, let's create one
desktop-file-install --vendor="fedora" \
--dir=$RPM_BUILD_ROOT/%{_datadir}/applications \
%{SOURCE1}
%check
# We don't test GUI tests.
#./Build test
%clean
rm -rf $RPM_BUILD_ROOT
%files
%defattr(-,root,root,-)
%doc Changes README
/usr/bin/padre
%{perl_vendorlib}/*
%{_datadir}/padre
%{_datadir}/applications/fedora-Padre.desktop
%{_mandir}/man3/*
%changelog
* Fri Oct 10 2008 Marcela Mašláňová <mmaslano(a)redhat.com> 0.10-4
- move templates into _datadir
* Fri Oct 10 2008 Marcela Mašláňová <mmaslano(a)redhat.com> 0.10-3
- add desktop file
* Fri Oct 10 2008 Marcela Mašláňová <mmaslano(a)redhat.com> 0.10-2
- fix review
- example and test files should have permission 644
- desktop file is missing in upstream version
* Fri Oct 03 2008 Marcela Mašláňová <mmaslano(a)redhat.com> 0.10-1
- Specfile autogenerated by cpanspec 1.77.
Index: .cvsignore
===================================================================
RCS file: /cvs/pkgs/rpms/perl-Padre/devel/.cvsignore,v
retrieving revision 1.1
retrieving revision 1.2
diff -u -r1.1 -r1.2
--- .cvsignore 15 Oct 2008 09:59:35 -0000 1.1
+++ .cvsignore 15 Oct 2008 10:21:05 -0000 1.2
@@ -0,0 +1 @@
+Padre-0.10.tar.gz
Index: sources
===================================================================
RCS file: /cvs/pkgs/rpms/perl-Padre/devel/sources,v
retrieving revision 1.1
retrieving revision 1.2
diff -u -r1.1 -r1.2
--- sources 15 Oct 2008 09:59:35 -0000 1.1
+++ sources 15 Oct 2008 10:21:05 -0000 1.2
@@ -0,0 +1 @@
+d036a33e475a962415e4ef9837c7a74e Padre-0.10.tar.gz
15 years, 6 months
[Bug 226285] Merge Review: perl-XML-Grove
by Red Hat Bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=226285
--- Comment #25 from Parag AN(पराग) <panemade(a)gmail.com> 2008-10-14 23:52:16 EDT ---
Ok. I will like to say only that I don't mind if anyone add me in CC to any
package review. I will be happy to help them. But as you all see what happened
here after I requested updated SRPM many times. So I gave up with conclusion no
one cares here so why should I keep myself in CC also? Let me work on other
reviews.
As I am working on some other things please allow me to stay away from this
review. I know Patrice will help this package to review.
Thanks all.
--
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.
15 years, 6 months