Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 546122 - metadata/news/2013-06-01-mysql-pbxt-dropped references invalid atoms
Summary: metadata/news/2013-06-01-mysql-pbxt-dropped references invalid atoms
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: [OLD] Unspecified (show other bugs)
Hardware: All Linux
: Normal normal (vote)
Assignee: Gentoo Linux MySQL bugs team
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2015-04-10 07:37 UTC by fkater
Modified: 2015-10-25 13:29 UTC (History)
5 users (show)

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


Attachments
0001-Fix-PMS-GLEP-42-compliance-of-2013-06-01-mysql-pbxt-.patch (0001-Fix-PMS-GLEP-42-compliance-of-2013-06-01-mysql-pbxt-.patch,1.72 KB, patch)
2015-09-13 10:46 UTC, Julian Ospald
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description fkater 2015-04-10 07:37:09 UTC
Since some days, with each portage update I get a lengthy warning about
something like a syntax violation in this metadata news:

metadata/news/2013-06-01-mysql-pbxt-dropped

I am using paludis/cave which can not seldomly considered as another
validator for the portage specifications. cave complains that in
the news file the package/use dependency specifcations like [use,use]
given for the below packages are invalid:

dev-db/mysql[pbxt,-minimal]
dev-db/mysql-cluster[pbxt,-minimal]
dev-db/mariadb[pbxt,-minimal]
dev-db/mariadb-galera[pbxt,-minimal]
dev-db/percona-server[pbxt,-minimal]
dev-db/google-mysql[pbxt,-minimal]

Please check if in this news file the specification is violated
or not. If not I'll pass this issue to the paludis people.
Comment 1 Michael Palimaka (kensington) gentoo-dev 2015-04-16 11:32:29 UTC
I guess the warning is because the pbxt USE flag has since disappeared.
Comment 2 fkater 2015-04-18 20:30:40 UTC
Indeed, the USE flag pbxt is gone since 09 Aug 2014, according to
portage/profiles/base/ChangeLog.

I don't know if the mentioned news from 2013 referencing that USE flag
could then simply be deleted -- but at least it would be nice to move that
reference to pbxt into a comment only line or similar, so that people
might be able to read it but interpreters like paludis/cave do not
complain anymore, causing 3 displays full of repeated warnings each time...
Comment 3 Michael Palimaka (kensington) gentoo-dev 2015-04-19 15:14:41 UTC
Sounds like Paludis being needlessly pedantic as usual. There's no requirement for the Display-If-Installed atom to still exist in the tree, and indeed there's valid use cases for that.
Comment 4 bitozoid 2015-05-27 09:42:00 UTC
I'm also a paludis user and I'm also getting lots of warnings.

According to
https://wiki.gentoo.org/wiki/GLEP:42#News_Item_Headers
'Display-If-Installed' requires an atom.

According to
https://dev.gentoo.org/~zmedico/portage/doc/man/portage.5.html
an atom does not come with USE flag.

I think it should be fixed or removed.
Comment 5 Julian Ospald 2015-09-12 19:53:52 UTC
(In reply to bitozoid from comment #4)
> I'm also a paludis user and I'm also getting lots of warnings.
> 
> According to
> https://wiki.gentoo.org/wiki/GLEP:42#News_Item_Headers
> 'Display-If-Installed' requires an atom.
> 
> According to
> https://dev.gentoo.org/~zmedico/portage/doc/man/portage.5.html
> an atom does not come with USE flag.
> 

Correct, but not only that... there is no information on EAPIs whatsoever in GLEP-42, so no particular EAPI can be assumed. USE Dependencies have been added in EAPI-2.

There are two ways:
* we just fix the news item
* we bump "News-Item-Format" to 2.0 and document in GLEP-42 that this version may use a full dependency spec with USE Dependencies

Since this bug has been open for months, I will just fix the news item in 2 weeks if there is no reply.
Comment 6 Ulrich Müller gentoo-dev 2015-09-12 21:40:04 UTC
(In reply to Julian Ospald (hasufell) from comment #5)
> (In reply to bitozoid from comment #4)
> > According to
> > https://dev.gentoo.org/~zmedico/portage/doc/man/portage.5.html
> > an atom does not come with USE flag.
> 
> Correct, but not only that...

No, I don't think this is correct. PMS uses "atom" as a synonym for "package dependency specification" which is defined here:
https://projects.gentoo.org/pms/5/pms.html#x1-820008.2.6

This agrees with the definition given in ebuild(5), and it can include SLOT or USE dependencies for EAPIs supporting them.

> there is no information on EAPIs whatsoever in
> GLEP-42, so no particular EAPI can be assumed.

Right.

> There are two ways:
> * we just fix the news item
> * we bump "News-Item-Format" to 2.0 and document in GLEP-42 that this
> version may use a full dependency spec with USE Dependencies

Bumping the format isn't really an option for a news item released in 2013. Users for whom the item would be still relevant cannot be expected to have the newest version of the package manager (which would be required for reading the new format) installed.
Comment 7 Julian Ospald 2015-09-12 22:34:38 UTC
(In reply to Ulrich Müller from comment #6)
> 
> Bumping the format isn't really an option for a news item released in 2013.
> Users for whom the item would be still relevant cannot be expected to have
> the newest version of the package manager (which would be required for
> reading the new format) installed.

So do you suggest we just fix the news item by removing the USE dependencies?
Comment 8 Ulrich Müller gentoo-dev 2015-09-13 08:36:17 UTC
(In reply to Julian Ospald (hasufell) from comment #7)
> So do you suggest we just fix the news item by removing the USE dependencies?

Obviously.
Comment 9 Julian Ospald 2015-09-13 10:46:11 UTC
Created attachment 411790 [details, diff]
0001-Fix-PMS-GLEP-42-compliance-of-2013-06-01-mysql-pbxt-.patch
Comment 11 Ulrich Müller gentoo-dev 2015-09-13 18:20:20 UTC
(In reply to Julian Ospald (hasufell) from comment #10)
> since no one seems to object:
> https://gitweb.gentoo.org/data/gentoo-news.git/commit/
> ?id=a0af9089c9a3f9f0789e67e3e028cda04e198a05

Reopening, you must also increase "Revision:" and re-sign the item.
Comment 12 Ulrich Müller gentoo-dev 2015-09-13 18:26:04 UTC
(In reply to Ulrich Müller from comment #11)
> [...] re-sign the item.

I just notice that it wasn't signed in the first place, so not your fault. It should be signed, though.
Comment 13 Julian Ospald 2015-09-13 18:38:44 UTC
sorry, missed that, should be fixed now