Overview

Request 685629 accepted

Qt 5.12.2


Jan Engelhardt's avatar

Please do not add new %{__make} (or other %__), and instead use make directly.


Fabian Vogt's avatar
author reviewer source maintainer

Why is that an issue? I had to manually expand %make_jobs, so it needs to be identical.


Jan Engelhardt's avatar

It's not really an issue, but %__make is always make on openSUSE anyway, so it felt kinda redundant.


Fabian Vogt's avatar
author reviewer source maintainer

I'll leave the review open for now - but not because of _make. There's a fix for plasma-framework that needs to be merged with Qt 5.12.2. Edits binary files though, so not directly suitable as patch.

Request History
Fabian Vogt's avatar

Vogtinator created request

Qt 5.12.2


Factory Auto's avatar

factory-auto added opensuse-review-team as a reviewer

Please review sources


Factory Auto's avatar

factory-auto accepted review

Check script succeeded


Saul Goodman's avatar

licensedigger accepted review

ok


Jan Engelhardt's avatar

jengelh added Vogtinator as a reviewer

%__make


Dominique Leuenberger's avatar

dimstar_suse set openSUSE:Factory:Staging:E as a staging project

Being evaluated by staging project "openSUSE:Factory:Staging:E"


Dominique Leuenberger's avatar

dimstar_suse accepted review

Picked openSUSE:Factory:Staging:E


Dominique Leuenberger's avatar

dimstar_suse changed priority to moderate => important

raising priority for openSUSE:Factory:Staging:E


Dominique Leuenberger's avatar

dimstar accepted review


Fabian Vogt's avatar

Vogtinator accepted review


Dominique Leuenberger's avatar

dimstar_suse accepted review

ready to accept


Dominique Leuenberger's avatar

dimstar_suse approved review

ready to accept


Dominique Leuenberger's avatar

dimstar_suse accepted request

Accept to openSUSE:Factory

openSUSE Build Service is sponsored by