FAQ Search Today's Posts Mark Forums Read
» Video Reviews

» Linux Archive

Linux-archive is a website aiming to archive linux email lists and to make them easily accessible for linux users/developers.


» Sponsor

» Partners

» Sponsor

Go Back   Linux Archive > Gentoo > Gentoo Development

 
 
LinkBack Thread Tools
 
Old 06-10-2010, 07:27 PM
Jeremy Olexa
 
Default gentoo-x86 commit in dev-python/traits: traits-3.4.0.ebuild

On Thu, 10 Jun 2010 19:19:33 +0000 (UTC), "Arfrever Frehtes Taifersar
Arahesis (arfrever)" <arfrever@gentoo.org> wrote:
> arfrever 10/06/10 19:19:33
>
> Modified: traits-3.4.0.ebuild
> Log:
> Fix dependencies. Use -fno-strict-aliasing.
> (Portage version: HEAD/cvs/Linux x86_64)
>
> Revision Changes Path
> 1.3 dev-python/traits/traits-3.4.0.ebuild

I see no reason to *not* add a ChangeLog entry here. Please, consider
the fact that some people don't want to go digging in cvs log to see who
added -fno-strict-aliasing or why. (as an example. I, personally, don't
care about -fno-strict-aliasing)

-Jeremy
 
Old 06-10-2010, 08:00 PM
Arfrever Frehtes Taifersar Arahesis
 
Default gentoo-x86 commit in dev-python/traits: traits-3.4.0.ebuild

2010-06-10 21:27:40 Jeremy Olexa napisał(a):
> I see no reason to *not* add a ChangeLog entry here.

ChangeLog entries are not required for trivial changes.

--
Arfrever Frehtes Taifersar Arahesis
 
Old 06-10-2010, 08:16 PM
Fabian Groffen
 
Default gentoo-x86 commit in dev-python/traits: traits-3.4.0.ebuild

On 10-06-2010 22:00:19 +0200, Arfrever Frehtes Taifersar Arahesis wrote:
> 2010-06-10 21:27:40 Jeremy Olexa napisał(a):
> > I see no reason to *not* add a ChangeLog entry here.
>
> ChangeLog entries are not required for trivial changes.

Apart from whether or not this is true, do I understand well that you
consider masqueraded EAPI bumps as homepage updates as trivial changes
too? (traitsbackendwx)


--
Fabian Groffen
Gentoo on a different level
 
Old 06-10-2010, 08:25 PM
Arfrever Frehtes Taifersar Arahesis
 
Default gentoo-x86 commit in dev-python/traits: traits-3.4.0.ebuild

2010-06-10 22:16:23 Fabian Groffen napisał(a):
> On 10-06-2010 22:00:19 +0200, Arfrever Frehtes Taifersar Arahesis wrote:
> > 2010-06-10 21:27:40 Jeremy Olexa napisał(a):
> > > I see no reason to *not* add a ChangeLog entry here.
> >
> > ChangeLog entries are not required for trivial changes.
>
> Apart from whether or not this is true, do I understand well that you
> consider masqueraded EAPI bumps as homepage updates as trivial changes
> too? (traitsbackendwx)

If a change in EAPI doesn't require any other changes in ebuild, then it's a trivial change.

--
Arfrever Frehtes Taifersar Arahesis
 
Old 06-10-2010, 08:45 PM
Arfrever Frehtes Taifersar Arahesis
 
Default gentoo-x86 commit in dev-python/traits: traits-3.4.0.ebuild

2010-06-10 22:20:44 Nirbheek Chauhan napisał(a):
> On Fri, Jun 11, 2010 at 1:30 AM, Arfrever Frehtes Taifersar Arahesis
> <Arfrever@gentoo.org> wrote:
> > 2010-06-10 21:27:40 Jeremy Olexa napisał(a):
> >> I see no reason to *not* add a ChangeLog entry here.
> >
> > ChangeLog entries are not required for trivial changes.
> >
>
> A "trivial" change is fixing a typo, or a manifest problem, a missing
> quotation mark, etc. Anything else is not "trivial".
>
> Anything that changes how an ebuild functions, what it does, or the
> installed files (and/or their contents) is NOT a trivial change.

This commit only removed some compiler warnings.

--
Arfrever Frehtes Taifersar Arahesis
 
Old 06-10-2010, 09:45 PM
Rémi Cardona
 
Default gentoo-x86 commit in dev-python/traits: traits-3.4.0.ebuild

Le 10/06/2010 22:45, Arfrever Frehtes Taifersar Arahesis a écrit :
> 2010-06-10 22:20:44 Nirbheek Chauhan napisał(a):
>> On Fri, Jun 11, 2010 at 1:30 AM, Arfrever Frehtes Taifersar Arahesis
>> <Arfrever@gentoo.org> wrote:
>>> 2010-06-10 21:27:40 Jeremy Olexa napisał(a):
>>>> I see no reason to *not* add a ChangeLog entry here.
>>>
>>> ChangeLog entries are not required for trivial changes.
>>>
>>
>> A "trivial" change is fixing a typo, or a manifest problem, a missing
>> quotation mark, etc. Anything else is not "trivial".
>>
>> Anything that changes how an ebuild functions, what it does, or the
>> installed files (and/or their contents) is NOT a trivial change.
>
> This commit only removed some compiler warnings.
>

Why argue about this? Just always add a ChangeLog entry, like everyone
else. This is in everyone's interest, including yours.

Cheers,

Rémi
 
Old 06-11-2010, 06:51 AM
Alex Alexander
 
Default gentoo-x86 commit in dev-python/traits: traits-3.4.0.ebuild

On Thu, Jun 10, 2010 at 11:45:14PM +0200, Rémi Cardona wrote:
> Le 10/06/2010 22:45, Arfrever Frehtes Taifersar Arahesis a écrit :
> > 2010-06-10 22:20:44 Nirbheek Chauhan napisał(a):
> >> On Fri, Jun 11, 2010 at 1:30 AM, Arfrever Frehtes Taifersar Arahesis
> >> <Arfrever@gentoo.org> wrote:
> >>> 2010-06-10 21:27:40 Jeremy Olexa napisał(a):
> >>>> I see no reason to *not* add a ChangeLog entry here.
> >>>
> >>> ChangeLog entries are not required for trivial changes.
> >>>
> >>
> >> A "trivial" change is fixing a typo, or a manifest problem, a missing
> >> quotation mark, etc. Anything else is not "trivial".
> >>
> >> Anything that changes how an ebuild functions, what it does, or the
> >> installed files (and/or their contents) is NOT a trivial change.
> >
> > This commit only removed some compiler warnings.
> >
>
> Why argue about this? Just always add a ChangeLog entry, like everyone
> else. This is in everyone's interest, including yours.

+1

besides, fixing dependencies or messing with cflags doesn't seem trivial
to me =]

--
Alex Alexander :: wired
Gentoo Developer
www.linuxized.com
 
Old 06-11-2010, 08:43 AM
Alexis Ballier
 
Default gentoo-x86 commit in dev-python/traits: traits-3.4.0.ebuild

On Thursday 10 June 2010 23:45:29 Arfrever Frehtes Taifersar Arahesis wrote:
[...]
> This commit only removed some compiler warnings.

This adds a cflag preventing the compiler to make some assumptions on the code
for its optimisations, hence making the code slower/bigger. While the
changelog entry will help figuring out why this was added, a comment in the
ebuild explaining why will be even better.
Once you'll have to dig into 4 years of cvs log to figure out wth a cflag
filtering/addition was added into an ebuild maybe you'll reconsider your
opinion.

Please always keep in mind that you're not the only one that will ever commit
to $package and even less the only one that will have a look at a given
ebuild.

A.
 
Old 06-12-2010, 03:47 PM
Jeroen Roovers
 
Default gentoo-x86 commit in dev-python/traits: traits-3.4.0.ebuild

On Thu, 10 Jun 2010 22:25:48 +0200
Arfrever Frehtes Taifersar Arahesis <Arfrever@gentoo.org> wrote:

> If a change in EAPI doesn't require any other changes in ebuild, then
> it's a trivial change.

Trivial is when you fix spelling in text sent to stdout/stderr or
change the text of a comment. Any change to code, including variables,
is not trivial and should be documented in the ChangeLog.


jer
 
Old 06-19-2010, 08:53 PM
Mike Frysinger
 
Default gentoo-x86 commit in dev-python/traits: traits-3.4.0.ebuild

On Thursday, June 10, 2010 16:45:29 Arfrever Frehtes Taifersar Arahesis wrote:
> 2010-06-10 22:20:44 Nirbheek Chauhan napisał(a):
> > On Fri, Jun 11, 2010 at 1:30 AM, Arfrever Frehtes Taifersar wrote:
> > > 2010-06-10 21:27:40 Jeremy Olexa napisał(a):
> > >> I see no reason to *not* add a ChangeLog entry here.
> > >
> > > ChangeLog entries are not required for trivial changes.
> >
> > A "trivial" change is fixing a typo, or a manifest problem, a missing
> > quotation mark, etc. Anything else is not "trivial".
> >
> > Anything that changes how an ebuild functions, what it does, or the
> > installed files (and/or their contents) is NOT a trivial change.
>
> This commit only removed some compiler warnings.

mucking with CFLAGS without documentation is wrong. compiler warnings come
and go, so a flag that was relevant one day could be completely extraneous the
next.

however, especially with strict aliasing, you arent "just fixing warnings",
you're changing optimization behavior of gcc to workaround broken C code.
this obviously does not fall anywhere near the "trivial" mark.

i see you still havent fixed this, so get on it already. a bug needs to be
opened somewhere to get the package properly *fixed*, and your change either
out right reverted or add a comment around the flag to indicate the open bug
on the issue.
append-flags ... #12345
-mike
 

Thread Tools




All times are GMT. The time now is 09:41 AM.

VBulletin, Copyright ©2000 - 2014, Jelsoft Enterprises Ltd.
Content Relevant URLs by vBSEO ©2007, Crawlability, Inc.
Copyright 2007 - 2008, www.linux-archive.org