Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 754696 - dev-php/jsonlint: QA issues
Summary: dev-php/jsonlint: QA issues
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All Linux
: Normal normal
Assignee: Thomas Deutschmann (RETIRED)
URL: https://qa-reports.gentoo.org/output/...
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2020-11-15 09:56 UTC by Michał Górny
Modified: 2020-11-15 16:05 UTC (History)
4 users (show)

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


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Michał Górny archtester Gentoo Infrastructure gentoo-dev Security 2020-11-15 09:56:51 UTC
dev-php/jsonlint
  MissingTestRestrict: version 1.8.2: missing RESTRICT="!test? ( test )" with IUSE=test


Please explain why are you introducing QA regressions and ignoring CI mail.
Comment 1 Thomas Deutschmann (RETIRED) gentoo-dev 2020-11-15 15:12:08 UTC
Because I just wasn't aware of that until today? 

> and ignoring CI mail

Can you please proof that claim? I didn't get *any* mail regarding this issue until this bug mail today.
Comment 2 Michał Górny archtester Gentoo Infrastructure gentoo-dev Security 2020-11-15 15:23:31 UTC
(In reply to Thomas Deutschmann from comment #1)
> Can you please proof that claim? I didn't get *any* mail regarding this
> issue until this bug mail today.

Here you are:

https://archives.gentoo.org/gentoo-automated-testing/message/2c9e3710643b81de096b5480749c4edf
Comment 3 Thomas Deutschmann (RETIRED) gentoo-dev 2020-11-15 15:32:38 UTC
Thank you, found that mail.

May I suggest that you will change subject for CI mails containing 'New warnings'? Sending these mails with a subject "Still has warning" seems wrong to me.
Comment 4 Larry the Git Cow gentoo-dev 2020-11-15 15:34:44 UTC
The bug has been closed via the following commit(s):

https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=cf9f180a59ff174f931cd487b1d0e2d6dc1641b5

commit cf9f180a59ff174f931cd487b1d0e2d6dc1641b5
Author:     Thomas Deutschmann <whissi@gentoo.org>
AuthorDate: 2020-11-15 15:34:14 +0000
Commit:     Thomas Deutschmann <whissi@gentoo.org>
CommitDate: 2020-11-15 15:34:39 +0000

    dev-php/jsonlint: fix MissingTestRestrict
    
    Closes: https://bugs.gentoo.org/754696
    Package-Manager: Portage-3.0.9, Repoman-3.0.2
    Signed-off-by: Thomas Deutschmann <whissi@gentoo.org>

 dev-php/jsonlint/jsonlint-1.8.2.ebuild | 1 +
 1 file changed, 1 insertion(+)
Comment 5 Michał Górny archtester Gentoo Infrastructure gentoo-dev Security 2020-11-15 16:05:15 UTC
(In reply to Thomas Deutschmann from comment #3)
> Thank you, found that mail.
> 
> May I suggest that you will change subject for CI mails containing 'New
> warnings'? Sending these mails with a subject "Still has warning" seems
> wrong to me.

Done.  It should deploy in an hour or so and then we'll see whether it works.