Bug 627012

Summary:	app-portage/flaggie-0.2.1 --sort mishandles semicolon USE flag syntax
Product:	Gentoo Linux	Reporter:	Georgy Yakovlev <gyakovlev>
Component:	Current packages	Assignee:	Michał Górny <mgorny>
Status:	RESOLVED UPSTREAM
Severity:	normal	CC:	jstein
Priority:	Normal
Version:	unspecified
Hardware:	All
OS:	Linux
Whiteboard:
Package list:		Runtime testing required:	---

Description Georgy Yakovlev archtester

2017-08-04 00:15:52 UTC

Here is quick case

> /etc/portage/package.use # flaggie --version
> flaggie 0.2.1`

and contets of a file
> /etc/portage/package.use # cat llvm
> sys-devel/clang LLVM_TARGETS: -NVPTX -AMDGPU
> sys-devel/llvm  LLVM_TARGETS: -NVPTX -AMDGPU

I run sort

> /etc/portage/package.use # flaggie --sort

and -AMDGPU gets moved in front of LLVM_TARGETS:

> /etc/portage/package.use # cat llvm
> sys-devel/clang -AMDGPU LLVM_TARGETS: -NVPTX
> sys-devel/llvm -AMDGPU LLVM_TARGETS: -NVPTX


I have not looked at the code myself yet, maybe that's something I can fix myself and submit a patch. Maybe not.

Comment 1 Georgy Yakovlev archtester

2017-08-04 03:55:39 UTC

I went to see the code and realized that there is an issue opened for that already
https://github.com/mgorny/flaggie/issues/13
what should I do with this bug?

Comment 2 Michał Górny archtester

2017-08-04 21:15:52 UTC

I track flaggie issues at GitHub. Long story short, the USE flag rework is WIP but I have so much on my plate that I don't even remember how far it's gotten. I'm sorry but please don't expect a complete fix anytime soon.

I think the current git (-9999) has a partial support for USE_EXPAND groups in package.use. However, I think I've disabled all the cleanup actions there as they don't support it yet.