php.net |  support |  documentation |  report a bug |  advanced search |  search howto |  statistics |  random bug |  login

go to bug id or search bugs for

Bug #73984 state stable/beta -> kidding?
Submitted: 2017-01-24 11:50 UTC Modified: 2017-02-01 23:21 UTC
From: spam2 at rhsoft dot net Assigned: danack (profile)
Status: Closed Package: imagick (PECL)
PHP Version: Irrelevant OS:
Private report: No CVE-ID: None
View Add Comment Developer Edit
 [2017-01-24 11:50 UTC] spam2 at rhsoft dot net 
Description:
------------
https://pecl.php.net/package/imagick

looks like somebody does not know what RC and stable/beta means or why are the release candidates stable and the last non-RC (3.4.2) is marked as beta?

3.4.3RC3 	
stable 	

3.4.3RC2 	
stable 	

3.4.3RC1 
stable

3.4.2 	
beta

Patches

Add a Patch

Pull Requests

Add a Pull Request

History

AllCommentsChangesGit/SVN commitsRelated reports
 [2017-01-24 13:27 UTC] cmb@php.net
-Status: Open +Status: Analyzed -Assigned To: +Assigned To: danack
 [2017-01-24 13:27 UTC] cmb@php.net 
Indeed, the <version> and <stability> elements of package.xml are
not in sync:
<https://github.com/mkoppanen/imagick/blob/3.4.3RC2/package.xml#L29-L36>.

Dan, can you please correct that?
 [2017-01-24 17:37 UTC] danack@php.net 
Yeah, I screwed up the stability on the last release, and the validation script that is meant to catch those errors didn't catch it. 

> Dan, can you please correct that?

The next release in a couple of days will have the correct stable/release number.

I've created a script to catch these errors in TravisCI (rather than the pecl upload validator), which should stop similar issues in the future.

Just as a reminder - if anyone who uses Imagick wants to help out with maintaining Imagick that help would be gratefully received: https://github.com/mkoppanen/imagick/issues/162
 [2017-01-24 17:50 UTC] spam2 at rhsoft dot net 
don't get me wrong but in that many months not a single time just look at the page because unconditional trust in some scripts is strange  - automatisms are nice but never replace human review, in no context of software development or server administration

otherwise the guy who last touched the pecl-rss-feed script would have realized that it points directly to the download tarballs for at least 2 years or so
 [2017-02-01 23:21 UTC] danack@php.net
-Status: Analyzed +Status: Closed
 [2017-02-01 23:21 UTC] danack@php.net 
The version/stability should be correct now, and the check is running on travis.

OP - you're right, it would be better for a human to check every release uploaded on pecl. Thanks for volunteering to do this. I look forward to reports where I've made a mistake and you've caught the error.
 
PHP Copyright © 2001-2024 The PHP Group
All rights reserved. 		Last updated: Sun May 05 14:01:31 2024 UTC