Bug 7429

Summary: [PATCH] gem_helper.rb is broken
Product: Mageia Reporter: Shlomi Fish <shlomif>
Component: RPM PackagesAssignee: D Morgan <dmorganec>
Status: RESOLVED FIXED QA Contact:
Severity: critical    
Priority: Normal CC: fundawang, luigiwalser, mageia, thierry.vignaud
Version: CauldronKeywords: PATCH
Target Milestone: ---   
Hardware: i586   
OS: Linux   
Whiteboard:
Source RPM: rpm-mageia-setup-1.158-1.mga3.src.rpm CVE:
Status comment:
Attachments: Patch to fix the problem.

Description Shlomi Fish 2012-09-10 16:00:45 CEST
Description of problem:

I'm going to attach a patch to fix some problems with /usr/lib/rpm/mageia/gem_helper.rb that got broken in the update to ruby-1.9.3 on Mageia Cauldron updates_testing.

Without it building the gem ruby-columnize (and probably other gems as well) yields an error.
Comment 1 Shlomi Fish 2012-09-10 16:02:03 CEST
Created attachment 2780 [details]
Patch to fix the problem.

This is a patch to fix the problem. Please apply it.
Comment 2 Shlomi Fish 2012-09-10 17:50:56 CEST
Assigning to Thierry Vignaud.

Assignee: bugsquad => thierry.vignaud

Comment 3 Thierry Vignaud 2012-09-10 17:54:46 CEST
See with Dexter Morgan, he is the one that commited pok's changes

Assignee: thierry.vignaud => dmorganec

Comment 4 David Walser 2012-09-10 18:05:12 CEST
(In reply to comment #3)
> See with Dexter Morgan, he is the one that commited pok's changes

Those were to rpm, not rpm-mageia-setup, and just fixed things, not caused breakage.  Building ruby gem packages should be simpler and compatible with mdv now.  It certainly wouldn't have anything to do with what changed in ruby 1.9.3.

CC: (none) => luigiwalser

Comment 5 Shlomi Fish 2012-09-10 18:45:45 CEST
Hi,

(In reply to comment #4)
> (In reply to comment #3)
> > See with Dexter Morgan, he is the one that commited pok's changes
> 
> Those were to rpm, not rpm-mageia-setup, and just fixed things, not caused
> breakage.  Building ruby gem packages should be simpler and compatible with mdv
> now.  It certainly wouldn't have anything to do with what changed in ruby
> 1.9.3.

So what should we do now? Who will commit the patch to the svn, release a new version and prepare a new package?

Regards,

-- Shlomi Fish
Comment 6 Colin Guthrie 2012-09-10 20:19:58 CEST
I think it's probably OK for you to update the code in the /soft/ svn tree as needed Shlomi, and then issue a release yourself.

Funda did this just the other day, so here is a (semi) template of what to do with his commits as examples (NOTE, he never submitted his build to the BS, so you should also test it and make sure his changes are sensible and valid. You should also double check any commits done since the previous release generally just in case someone has made some bogus changes, committed them and never pushed to production. If you find problems you should mock the committer mercilessly until they cry, or just send them a friendly pointer/query if you prefer! :p).


1. First commit the code changes needed. Make sure to reference where the patch came from if appropriate (i.e. if it came from Mdv, please say so). Also do not forget to bump the version in the appropriate places and update the news file.
http://svnweb.mageia.org/soft?view=revision&revision=5759

2. Build a tarball. I'm not sure of the exact procedure here but it's typically a matter of running ./autogen.sh and running "make dist".

3. Update the package and make sure the new tarball works as expected and you can install your updated package and it solves the bug. See here: http://svnweb.mageia.org/packages?view=revision&revision=289992 Note: don't commit this change yet.

4. Tag the svn release. This doesn't seem to have been done of late, so I wouldn't worry too much about it.

5. Now this is "official", you can commit the package change made in 3, upload the binary via mgarepo and and submit it for build.


When changing RPM stuff it's quite easy to break the build system which requires sysadmin intervention. So testing is key here, and blind submits will be very much frowned upon :)


If you do anything wrong, I'm sure the resident guardian of commits, Thierry, will notice and give you a friendly pointer (or ridicule you in public depending on the severity of the mistake!) :)

Hope that helps.

Col

CC: (none) => mageia

Comment 7 Shlomi Fish 2012-09-10 21:43:52 CEST
(In reply to comment #6)
> I think it's probably OK for you to update the code in the /soft/ svn tree as
> needed Shlomi, and then issue a release yourself.

Thanks, Colin!

I did all that and now my changes are in Cauldron core/updates_testing.

Regards,

-- Shlomi Fish
Thierry Vignaud 2012-10-24 01:19:20 CEST

CC: (none) => fundawang, thierry.vignaud

Thierry Vignaud 2012-10-24 01:21:53 CEST

Keywords: (none) => PATCH

Comment 8 Thierry Vignaud 2012-12-06 09:14:08 CET
Fixed since 1.159

Status: NEW => RESOLVED
Resolution: (none) => FIXED