Bug 225087 - devel/pecl-weakref: portscout failure: Bad limitw value provided by portconfig variable in port devel/pecl-weakref
Summary: devel/pecl-weakref: portscout failure: Bad limitw value provided by portconfi...
Status: Closed FIXED
Alias: None
Product: Ports & Packages
Classification: Unclassified
Component: Individual Port(s) (show other bugs)
Version: Latest
Hardware: Any Any
: --- Affects Only Me
Assignee: Jochen Neumeister
URL:
Keywords: needs-patch
Depends on:
Blocks:
 
Reported: 2018-01-11 22:53 UTC by Yuri Victorovich
Modified: 2018-02-25 18:06 UTC (History)
2 users (show)

See Also:
bugzilla: maintainer-feedback? (joneum)


Attachments
v1 (381 bytes, patch)
2018-02-14 06:14 UTC, Tobias C. Berner
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Yuri Victorovich freebsd_committer freebsd_triage 2018-01-11 22:53:41 UTC
Portscout printed this:
> Bad limitw value provided by portconfig variable in port devel/pecl-weakref

Please fix!

Thanks,
Yuri
Comment 1 Yuri Victorovich freebsd_committer freebsd_triage 2018-01-12 06:20:37 UTC
Why is this not a bug?
Comment 2 Yuri Victorovich freebsd_committer freebsd_triage 2018-01-12 22:56:00 UTC
Sorry,

But this looks like a bug to me.
portscout complains about the PORTSCOUT tag.
The tag is places solely for the portscout consumption, and it doesn't like it. :-)

Yuri
Comment 3 Tobias C. Berner freebsd_committer freebsd_triage 2018-02-10 09:42:13 UTC
Good catch, 

Joneum, can you fix it:
https://github.com/freebsd/portscout/blob/master/docs/portscout-portconfig.txt#L28


mfg Tobias
Comment 4 Jochen Neumeister freebsd_committer freebsd_triage 2018-02-10 09:53:44 UTC
@reporter: please upload a patch when you open a PR. Thanks
Comment 5 Yuri Victorovich freebsd_committer freebsd_triage 2018-02-10 10:01:36 UTC
Jochen,

You can't expect reporters to send fixes for every problem they find.
Finding a problem and reporting it is already a contribution.
You as a maintainer should know best how to fix problems in your ports.

Regards,
Yuri
Comment 6 Jochen Neumeister freebsd_committer freebsd_triage 2018-02-10 10:04:44 UTC
What exactly is it that bothers you? Why does it bother you that I get PORTSCOUT wrong? Do you always open PRs without a patch?
Comment 7 Yuri Victorovich freebsd_committer freebsd_triage 2018-02-10 10:07:48 UTC
(In reply to Jochen Neumeister from comment #6)

Nothing bothers me at all, thanks for asking.
I just reported a bug, that's all.

Yuri
Comment 8 Yuri Victorovich freebsd_committer freebsd_triage 2018-02-10 10:15:57 UTC
Section 5.5. MAINTAINER of the Porters Handbook says:
> A maintainer volunteers to keep a port in good working order. Maintainers have the primary responsibility for their ports, but not exclusive ownership.

According to this, you have the primary responsibility, not submitters.

https://www.freebsd.org/doc/en/books/porters-handbook/makefile-maintainer.html
Comment 9 Jochen Neumeister freebsd_committer freebsd_triage 2018-02-10 10:39:53 UTC
and i say, PORTSCOUT is not a bug.

I will wait 2 weeks for a patch, and is there nothing, i closed here
Comment 10 Tobias C. Berner freebsd_committer freebsd_triage 2018-02-10 11:27:04 UTC
Moin moin 

Jochen, limitw is to limit between odd and even version numbers for example if the odd ones are always 'experimental' releases, you can make portscout only complain if there are new even versions inbound. 
I think for example Gnome uses that scheme, where 3.20 is stable and 3.21 is preview. 

Portscout has 'limitw:<position>:<even/oddness>'


Now, what you want is to tell portscout, it should only bug you if there is a 0.x.y available with a leading 0.

For this, portscout has "limit:<version-regex>". 


The regex you have already looks correct, but you used the wrong portscout limit-variable -- if you switch it from 'limitw' to 'limit', everything should work as it should [tm].




mfg Tobias
Comment 11 Jochen Neumeister freebsd_committer freebsd_triage 2018-02-10 11:30:51 UTC
Then Yuri has the necessary info for the patch, because it bothers him
Comment 12 Tobias C. Berner freebsd_committer freebsd_triage 2018-02-14 06:14:37 UTC
Created attachment 190603 [details]
v1
Comment 13 commit-hook freebsd_committer freebsd_triage 2018-02-25 18:06:02 UTC
A commit references this bug:

Author: joneum
Date: Sun Feb 25 18:05:52 UTC 2018
New revision: 462972
URL: https://svnweb.freebsd.org/changeset/ports/462972

Log:
  fix portscout

  PR:		225087
  Reported by:	yuri

Changes:
  head/devel/pecl-weakref/Makefile