|Summary:||Merge Review: perl-PDL|
|Product:||[Fedora] Fedora||Reporter:||Nobody's working on this, feel free to take it <nobody>|
|Component:||Package Review||Assignee:||Orion Poplawski <orion>|
|Status:||CLOSED NEXTRELEASE||QA Contact:||Fedora Package Reviews List <fedora-package-review>|
|Fixed In Version:||Doc Type:||Bug Fix|
|Doc Text:||Story Points:||---|
|Last Closed:||2008-01-30 17:53:29 UTC||Type:||---|
|oVirt Team:||---||RHEL 7.3 requirements from Atomic Host:|
|Cloudforms Team:||---||Target Upstream Version:|
Description Nobody's working on this, feel free to take it 2007-01-31 20:37:27 UTC
Fedora Merge Review: perl-PDL http://cvs.fedora.redhat.com/viewcvs/devel/perl-PDL/ Initial Owner: firstname.lastname@example.org
Comment 1 Orion Poplawski 2007-07-31 21:55:14 UTC
rpmlint: W: perl-PDL macro-in-%changelog description - can be ignored W: perl-PDL mixed-use-of-spaces-and-tabs (spaces: line 1, tab: line 3) - Fixed in patch I think the Summary should be changed to something like "The Perl Data Language". - package meets naming guidelines - package meets packaging guidelines - license (GPL/Artistic) OK, text in %doc, matches source - spec file legible, in am. english - source matches upstream - package compiles on devel (x86) - no unnecessary BR - no locales - not relocatable - owns all directories that it creates - no duplicate files - permissions ok - %clean ok - macro use consistent - code, not content - no need for -docs - nothing in %doc affects runtime - no need for .desktop file Consider including some more files in %doc: BUGS Changes Known_problems Release_Notes A number of PDL modules are not being built that could be. I'll be attaching a number of patches that enable them as well as getting PDL to build for devel again.
Comment 2 Orion Poplawski 2007-07-31 21:58:22 UTC
Created attachment 160366 [details] patch to devel Patch to spec and perl-PDL-settings.patch that: - turns on 3D/GL - turns on IO Browser - changes tabs to spaces - adds a bunch of BRs to enable more modules - removes unneeded Provides
Comment 3 Orion Poplawski 2007-07-31 21:59:06 UTC
Created attachment 160367 [details] patch to look for hdf devel files in the right place
Comment 4 Orion Poplawski 2007-07-31 21:59:37 UTC
Created attachment 160368 [details] patch to fix some tests
Comment 5 Orion Poplawski 2007-07-31 22:00:09 UTC
Created attachment 160369 [details] patch to find 64-bit libraries for certain modules
Comment 6 Orion Poplawski 2007-08-01 15:47:47 UTC
Looks like we also need a BR: libGLU-devel
Comment 7 Orion Poplawski 2007-08-01 16:10:52 UTC
Make that BR: libGLU-devel libGL-devel - though GLU depended on GL, but I guess not.
Comment 8 Orion Poplawski 2007-08-01 17:40:08 UTC
Created attachment 160436 [details] Remove -lXext library Scratch the BR libGL-devel, real fix is to remove -lXext from the GL linking options.
Comment 9 Robin Norwood 2007-08-06 17:33:12 UTC
ok, applied your patches, added the additional files to %docs, fixed the 'unescaped % in %changelog' warning, and a non-UTF-8 PDL::Complex.3pm. However, rpmlint is still complaining about: W: perl-PDL unversioned-explicit-provides perl(PDL::Config) etc... And company. Wouldn't it be better to version these with the same version as perl-PDL? also rpmlint complains about: W: perl-PDL devel-file-in-non-devel-package /usr/lib/perl5/vendor_perl/5.8.8/i386-linux-thread-multi/PDL/Core/pdlsimple.h etc... We could probably move these to a perl-PDL-devel package, if they aren't needed at runtime.
Comment 10 Orion Poplawski 2007-08-09 20:09:14 UTC
(In reply to comment #9) > > However, rpmlint is still complaining about: > > W: perl-PDL unversioned-explicit-provides perl(PDL::Config) > etc... > > And company. Wouldn't it be better to version these with the same version as > perl-PDL? Well, the vast majority of the automatic perl provides are unversioned too, so my take would be to leave the version out for now. > also rpmlint complains about: > > W: perl-PDL devel-file-in-non-devel-package > /usr/lib/perl5/vendor_perl/5.8.8/i386-linux-thread-multi/PDL/Core/pdlsimple.h > etc... > > We could probably move these to a perl-PDL-devel package, if they aren't needed > at runtime. > Not according to: http://fedoraproject.org/wiki/PackagingDrafts/Perl#head-b867c6067f009c5ea8f21723d028e8b40fae15c7 Some more stuff: - I've got perl-ExtUtils-F77 added to Fedora, so we can add: BuildRequires: perl(ExtUtils::F77) BuildRequires: gcc-gfortran I'll also attach a patch that uses the RPM_OPT_FLAGS and -fPIC to compile to fortran code, and fixes a test for gfortran/f77 compatibility. This allows PDL::Slatec to build, so we can remove that from the listed Provides. - Still need BuildRequires: libGLU-devel - Looks like the License should be re-examined and the tag updated. Possibly "GPL+ or Artistic" though some files seem a little different. - Looks like we need to add Provides: perl(PDL::Graphics::TriD::Objects) - We need to filter perl(Win32::DDE::Client) from Requires now. - Don't forget to bump release and add changelog entries! I'll attach a patch for my spec changes as well.
Comment 11 Orion Poplawski 2007-08-09 20:11:03 UTC
Created attachment 161017 [details] Patch to spec
Comment 12 Orion Poplawski 2007-08-09 20:11:48 UTC
Created attachment 161018 [details] patch to fix fortran issues
Comment 13 Robin Norwood 2007-08-10 16:11:54 UTC
Awesome. Applied your patch, except for the change to the License field. I've pinged Tom Callaway to take a look at the license first. Looks like the standard 'same terms as perl itself' to me - plus the one public domain image.
Comment 14 Robin Norwood 2007-08-10 16:56:07 UTC
Tom confirmed that you are correct about the license issue, I'll fix it. Also, it is failing to build in the buildsystem. I'll take a look and try to fix it today.
Comment 15 Orion Poplawski 2007-08-14 21:10:42 UTC
Created attachment 161309 [details] Updated test patch Looks like ppc is pickier about test output for some reason. This updated patch shuts up xvals.t some more. With this change it builds for me on ppc and ppc64.
Comment 16 Robin Norwood 2007-08-15 14:59:19 UTC
Excellent, thanks. 2.4.3-4 should be in rawhide now. Please take a look. By the by, are any of these patches submitted upstream? Looks like some of them would be appropriate.
Comment 17 Orion Poplawski 2007-08-16 18:59:20 UTC
Looks good to me. APPROVED. I haven't sent any patches upstream, I haven't yet managed to subscribe to PDL-porters. Looks like that there are similar patches for some of this issues in upstream CVS. I would be nice if they did a new release, it's been almost a full year since the last one. I'd love to see this make it into EPEL for EL-5 at least.
Comment 18 Robin Norwood 2007-08-16 20:17:07 UTC
"I'd love to see this make it into EPEL for EL-5 at least." If someone were to file a bug for that, I'd be happy to take a look at it. I'm guessing the deps will be a pain, though.
Comment 19 Orion Poplawski 2008-01-29 16:39:09 UTC
Created attachment 293304 [details] patch to support EL-5 GL lib All necessary dependencies for perl-PDL are now in EPEL-5. The attached patch is necessary to handle the version of GL in EL-5. With this patch, it builds fine for me.
Comment 20 Robin Norwood 2008-01-29 19:00:23 UTC
Package Change Request ====================== Package Name: perl-PDL New Branches: EL-5 There we go. Orion, would you like to maintain or co-maintain this package in EPEL and/or Fedora? I don't actually use it, so you'll probably do a much better job than I at maintaining it.
Comment 21 Orion Poplawski 2008-01-29 19:04:58 UTC
(In reply to comment #20) > Orion, would you like to maintain or co-maintain this package in EPEL and/or > Fedora? I don't actually use it, so you'll probably do a much better job than I > at maintaining it. Any/all would be fine.
Comment 22 Robin Norwood 2008-01-29 19:50:37 UTC
Ok, here's a new request with that change, too: Package Change Request ====================== Package Name: perl-PDL New Branches: EL-5 Updated Fedora Owners: rnorwood, orion Updated EPEL Owners: rnorwood, orion
Comment 23 Kevin Fenzi 2008-01-30 17:09:20 UTC
Comment 24 Robin Norwood 2008-01-30 17:20:17 UTC
ok, building for EL5
Comment 25 Robin Norwood 2008-01-30 17:22:20 UTC
oops, looks like orion beat me to it. All yours. :-)
Comment 26 Orion Poplawski 2008-01-30 17:53:29 UTC
Build succeeded. Closing again. Thanks all!
Comment 27 Petr Pisar 2012-06-04 14:29:15 UTC
Package Change Request ====================== Package Name: perl-PDL New Branches: Owners: InitialCC: perl-sig Please add perl-sig user with watch* permissions only to all Fedora branches.
Comment 28 Gwyn Ciesla 2012-06-04 14:34:36 UTC