Bug 153236

Summary: mod_perl: update and specfile cleanup
Product: [Fedora] Fedora Reporter: Jose Pedro Oliveira <jose.p.oliveira.oss>
Component: mod_perlAssignee: Joe Orton <jorton>
Status: CLOSED RAWHIDE QA Contact: David Lawrence <dkl>
Severity: medium Docs Contact:
Priority: medium    
Version: rawhideCC: redhat-bugzilla, wtogami
Target Milestone: ---   
Target Release: ---   
Hardware: All   
OS: Linux   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2005-04-27 09:41:32 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Bug Depends On:    
Bug Blocks: 136450    
Attachments:
Description Flags
Update and specfile cleanup
none
Full specfile
none
mod_perl 2.0.0 rc5 specfile patch
none
Filter requirements update none

Description Jose Pedro Oliveira 2005-04-03 21:24:37 UTC
Version-Release number of selected component (if applicable):
1.99_17-2

Expected results:
2.0.0-RC4

Additional info:
Patch included.

Known problems:
- the test suite is protected by a non-existent arch (%ifarch 1386)
- the test suite fails (mod_perl 1.99_17 and 2.0.0-rc4)
  I think is only file/directories permissions problems. I will try to
  look at it in a few days.

Related bug:
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=151178

Comment 1 Jose Pedro Oliveira 2005-04-03 21:24:38 UTC
Created attachment 112638 [details]
Update and specfile cleanup

Comment 2 Joe Orton 2005-04-04 13:05:06 UTC
Thanks Jose, any chance you could submit a patch without all the whitespace
changes - it makes it hard to review?

(I prefer non-vertically-aligned field values in spec files, anyway ;)

Comment 3 Michael Schwendt 2005-04-04 13:14:36 UTC
"diff -bu" (-b ignores changes in amount of white-space)

Comment 4 Jose Pedro Oliveira 2005-04-04 13:51:29 UTC
Created attachment 112649 [details]
Full specfile

Joe Orton,

Attaching the full specfile.  The specfile is using the Fedora.us and
Fedora.Extras template.

If you still want I can redo the specfile but only at the end of the afternoon
(5 hours).

jpo

Comment 5 Jose Pedro Oliveira 2005-04-06 16:54:27 UTC
Joe Orton,

Have you already made the update? Or do you still want me to rediff the patch?

jpo

Comment 6 Joe Orton 2005-04-11 13:23:39 UTC
Sorry for the delay, I was out for a couple of days.  It's not the most helpful
to attach full updated specfiles; the best thing to do is to attach a diff
against the /devel/ spec file from CVS (again, without all the whitespace
changes); this way I can review the changes easily.

Comment 7 Warren Togami 2005-04-14 20:44:52 UTC
Joe mind if I handle this report?  I'll personally review and test all changes.

Comment 8 Joe Orton 2005-04-14 20:57:05 UTC
Sure, go ahead!

Comment 9 Warren Togami 2005-04-17 06:18:11 UTC
http://people.redhat.com/wtogami/temp/mod_perl.spec
upgraded to RC5
excluded INSTALL because it is useless build instructions

http://people.redhat.com/wtogami/temp/mod_perl-555857-x86_64-crowe.devel.redhat.com.log
Everything seems good except lib64...

Comment 10 Jose Pedro Oliveira 2005-04-17 14:30:34 UTC
Created attachment 113290 [details]
mod_perl 2.0.0 rc5 specfile patch

Comment 11 Jose Pedro Oliveira 2005-04-17 14:36:44 UTC
Warren,

The mod_perl 2.0.0 rc5 build should also have failed in i386 as several files
have been renamed (Apache:: -> Apache2::).  Files in the new Apache2 namespace
weren't being included in the %files section.

Patch: previous comment


*Very important* note from the mod_perl 2.0.0 rc5 (1.999_22) changelog:
----------
1.999_22

    ******************** IMPORTANT ********************
    this version of mod_perl is completely incompatible
      with prior versions of mod_perl, both 1.XX and
     1.99_XX. Please read the below changes carefully.
    ***************************************************
   ...
----------
Source: http://search.cpan.org/~geoff/mod_perl-2.0.0-RC5/Changes

Can this warning be added to the FC4 release notes?

jpo



Comment 12 Warren Togami 2005-04-17 21:35:28 UTC
http://cvs.fedora.redhat.com/viewcvs/fedora-release/?root=fedora
/cvs/fedora is where you can check this out and fix it yourself, I think.


Comment 13 Jose Pedro Oliveira 2005-04-18 23:05:00 UTC
Created attachment 113349 [details]
Filter requirements update

Warren,

Trying to install mod_perl-2.0.0-0.rc5.1 gives the following messages:
---
  perl(Apache2::FunctionTable) is needed by mod_perl-2.0.0-0.rc5.1.i386
  perl(Apache2::StructureTable) is needed by mod_perl-2.0.0-0.rc5.1.i386
---

The filter needs to be updated as several perl modules were renamed in RC5.
The patch corrects the problem.

Comment 14 Jose Pedro Oliveira 2005-04-18 23:14:11 UTC
Ignore previous comment and patch.
Already in CVS (2.0.0-0.rc5.2).

Comment 15 Warren Togami 2005-04-27 09:41:32 UTC
Fixed in perl main package.