Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 144471 - Gaim 2.0 beta 3.1 (Version Bump Request)
Summary: Gaim 2.0 beta 3.1 (Version Bump Request)
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: New packages (show other bugs)
Hardware: All Linux
: High enhancement (vote)
Assignee: Gentoo Net-im project
URL: http://sourceforge.net/project/showfi...
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2006-08-19 17:13 UTC by Aaron Sheldon
Modified: 2006-09-05 21:44 UTC (History)
4 users (show)

See Also:
Package list:
Runtime testing required: ---


Attachments
version bump ebuild (gaim-2.0.0_beta3-r2.ebuild,6.29 KB, application/ebuild)
2006-08-28 06:30 UTC, Drew Winstel
Details
patch to include IUSE=doc (gaim-2.0.0_beta3-r2.ebuild.patch,1.06 KB, application/octet-stream)
2006-08-29 05:27 UTC, Drew Winstel
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Aaron Sheldon 2006-08-19 17:13:29 UTC
http://sourceforge.net/project/showfiles.php?group_id=235&package_id=253

Gaim 2.0 Beta 3.1 is available for download now on Sourceforge.
Comment 1 Drew Winstel 2006-08-21 13:33:05 UTC
The good news is a version bump ebuild works.  The bad news is that I have no idea what to call it since gaim-2.0.0_beta3.1 is non-canonical.

 # ebuild gaim-2.0.0_beta3.1.ebuild digest
!!! net-im/gaim-2.0.0_beta3.1 does not follow correct package syntax.

Any ideas?
Comment 2 M. Franco 2006-08-24 00:52:40 UTC
(In reply to comment #1)
> The good news is a version bump ebuild works.  The bad news is that I have no
> idea what to call it since gaim-2.0.0_beta3.1 is non-canonical.
> 
>  # ebuild gaim-2.0.0_beta3.1.ebuild digest
> !!! net-im/gaim-2.0.0_beta3.1 does not follow correct package syntax.
> 
> Any ideas?
> 

What about net-im/gaim-2.0.0_beta31 ?
beta 4 would be  net-im/gaim-2.0.0_beta40
Comment 3 ScytheMan 2006-08-25 10:06:11 UTC
(In reply to comment #2)
> (In reply to comment #1)
> > The good news is a version bump ebuild works.  The bad news is that I have no
> > idea what to call it since gaim-2.0.0_beta3.1 is non-canonical.
> > 
> >  # ebuild gaim-2.0.0_beta3.1.ebuild digest
> > !!! net-im/gaim-2.0.0_beta3.1 does not follow correct package syntax.
> > 
> > Any ideas?
> > 
> 
> What about net-im/gaim-2.0.0_beta31 ?
> beta 4 would be  net-im/gaim-2.0.0_beta40
> 
i would prefere net-im/gaim-2.0.0_beta3a

if a beta 4 will be released, than there would be no need to call the package beta40 
Comment 4 Drew Winstel 2006-08-25 10:21:54 UTC
(In reply to comment #3)
> i would prefere net-im/gaim-2.0.0_beta3a

As would I, but...
 # ebuild gaim-2.0.0_beta3a.ebuild digest
!!! net-im/gaim-2.0.0_beta3a does not follow correct package syntax.
Comment 5 Bill Weepie 2006-08-26 16:02:39 UTC
Why not just call the build gaim-2.0.0_beta3-r2?  You could add a note with the changelog saying that it is beta3.1.  Anyone clever enough to unmask the betas should know enough to read the changelogs too.
Comment 6 M. Franco 2006-08-27 23:49:53 UTC
We could also ask ourself if we should change the package syntax or not ?

Will we have other packages called *-beat#.# coming ? want we go this way ?
If not, I think -r2 would goes well.
Even if we find that changing the syntax is the way, it would take time to be accepted. I suggest going the -r2 way.

What's your opinion guys ?
Comment 7 Drew Winstel 2006-08-28 06:30:36 UTC
Created attachment 95279 [details]
version bump ebuild

The Gaim devs have always taken a bit of a renegade approach to naming; I don't see why this should be any different.  Perhaps a little more freedom in naming is necessary.  Here's the -r2 ebuild.
Comment 8 Lupo 2006-08-28 12:39:01 UTC
> is necessary.  Here's the -r2 ebuild.
 
Thank you for that ebuild. Unfortunately emerging this results in following error:
"!!! ERROR: net-im/gaim-2.0.0_beta3-r2 failed.
Call stack:
  ebuild.sh, line 1555:   Called dyn_setup
  ebuild.sh, line 668:   Called pkg_setup
  gaim-2.0.0_beta3-r2.ebuild, line 130:   Called die

!!! Configure failed"

Regards,
Lupo

Comment 9 Drew Winstel 2006-08-28 13:12:30 UTC
Could we see more info than just that?  

Also, what version of Portage are you using?
Comment 10 ScytheMan 2006-08-28 15:25:34 UTC
(In reply to comment #8)
> > is necessary.  Here's the -r2 ebuild.
> 
> Thank you for that ebuild. Unfortunately emerging this results in following
> error:
> "!!! ERROR: net-im/gaim-2.0.0_beta3-r2 failed.
> Call stack:
>   ebuild.sh, line 1555:   Called dyn_setup
>   ebuild.sh, line 668:   Called pkg_setup
>   gaim-2.0.0_beta3-r2.ebuild, line 130:   Called die
> 
> !!! Configure failed"
> 
> Regards,
> Lupo
> 
I compared this ebuild with my update of the gaim beta 3 ebuild (which is working for me, btw.), and these are the differences:

the -r2 Version in the bugzilla:
src_unpack() {
	unpack ${A}
	cd ${S}

	epatch ${FILESDIR}/${P}-as-needed.patch
	epatch ${FILESDIR}/${P}-avahi-compat.patch

	eautomake || die "Failed running eautomake"
}

my updated version:

src_unpack() {
	unpack ${A}
	cd ${S}

	epatch ${FILESDIR}/${PN}-2.0.0_beta3-as-needed.patch
	epatch ${FILESDIR}/${PN}-2.0.0_beta3-avahi-compat.patch

	eautomake || die "Failed running eautomake"
}

i don't know if this is good or bad what i've done, because my knowledge about ebuilds isn't very great...



Comment 11 Drew Winstel 2006-08-28 16:08:09 UTC
The way we both have it defined works, because the variable expansions are identical.  That is, "${PN}-2.0.0_beta3" == "${P}" as long as the ebuild is of the form gaim-2.0.0_beta3-r*.ebuild.

Which one is better is beyond me, though.  
Comment 12 Drew Winstel 2006-08-28 18:44:28 UTC
(In reply to comment #11)
> The way we both have it defined works, because the variable expansions are
> identical.  That is, "${PN}-2.0.0_beta3" == "${P}" as long as the ebuild is of
> the form gaim-2.0.0_beta3-r*.ebuild.
> 
> Which one is better is beyond me, though.  
> 

Scratch that, my way (${P}) is more future-proof, because your way will require a rewrite if and when beta4 comes out, or so I think... perhaps an expert can confirm that.
Comment 13 ScytheMan 2006-08-28 18:54:32 UTC
(In reply to comment #12)
> (In reply to comment #11)
> > The way we both have it defined works, because the variable expansions are
> > identical.  That is, "${PN}-2.0.0_beta3" == "${P}" as long as the ebuild is of
> > the form gaim-2.0.0_beta3-r*.ebuild.
> > 
> > Which one is better is beyond me, though.  
> > 
> 
> Scratch that, my way (${P}) is more future-proof, because your way will require
> a rewrite if and when beta4 comes out, or so I think... perhaps an expert can
> confirm that.
> 
my ebuild was has a _beta31 ending, so it wouldn't work with ${P} - so i used ${P}, i thought this was the fault when he emerged is, because the ebuilds are similar except this part and my ebuild works on my machine. my fault ;)
Comment 14 M. Franco 2006-08-29 00:36:22 UTC
(In reply to comment #7)
> Created an attachment (id=95279) [edit]
> version bump ebuild
> 
> The Gaim devs have always taken a bit of a renegade approach to naming; I don't
> see why this should be any different.  Perhaps a little more freedom in naming
> is necessary.  Here's the -r2 ebuild.
> 

I tried this ebuild and I only got an armless warning during compilation :
configure: WARNING: *** doxygen not found, docs will not be available

All fine for me
Comment 15 Drew Winstel 2006-08-29 05:27:54 UTC
Created attachment 95365 [details]
patch to include IUSE=doc

Here's a simple ebuild patch I threw together to enable support for the "doc" USE flag and squelch the warning if doc is not enabled.  Try it and let me know (I've only done preliminary testing).
Comment 16 M. Franco 2006-08-30 08:17:16 UTC
(In reply to comment #15)
> Created an attachment (id=95365) [edit]
> patch to include IUSE=doc
> 
> Here's a simple ebuild patch I threw together to enable support for the "doc"
> USE flag and squelch the warning if doc is not enabled.  Try it and let me know
> (I've only done preliminary testing).
> 

No more warnings about the doc.

Everything seems fine.
Comment 17 Olivier Crete (RETIRED) gentoo-dev 2006-09-05 17:55:02 UTC
Sorry for the delay, I've added the doc use flag, and added beta 3.1 as -r2. And thanks for doing all the work for me!
Comment 18 Drew Winstel 2006-09-05 21:44:11 UTC
(In reply to comment #17)
> Sorry for the delay, I've added the doc use flag, and added beta 3.1 as -r2.
> And thanks for doing all the work for me!
> 

Thanks!

Should -r1 be removed from Portage?