On 2/22/22 1:56 PM, Jason L Tibbitts III wrote:
Brandon Nielsen <nielsenb@xxxxxxxxxxx> writes:
I would like to see the forge macros removed from the guidelines if
development truly has ceased.
I would like to get into them and at least see what needs to change
but... the internal implementation is somewhat baroque and my time is
severely limited. I think there is good stuff there but I don't know
how much work would be required to salvage it.
One problem is that at least I thought that significant portions of the
Go tooling relied on it, and there's no alternative that has any kind of
automation.
- J<
Looking at this a little more, I'm not really sure we need to be
discouraging their use. The code[0] doesn't look that bad to me, and
there are only 2 issues open against it.
The fact nobody is stepping up to at least comment on the two issues is
a bit of a concern. The first one[1] looks really problematic, and I
don't see how you can fix it without coordinating with any package using
the broken behavior. Hopefully it isn't as bad as it looks?
The second one[2] is an RFE, and looks like one I may try to address if
nobody gets to it before I have some free time.
I do agree with the sentiment elsewhere in this thread that for the
majority of cases, it's easy enough to just work with the URLs directly.
Maybe the git forge macro documentation moves somewhere else instead?
[0] -
https://src.fedoraproject.org/rpms/redhat-rpm-config/blob/rawhide/f/forge.lua
[1] - https://bugzilla.redhat.com/show_bug.cgi?id=2048362
[2] - https://bugzilla.redhat.com/show_bug.cgi?id=2035935
_______________________________________________
devel mailing list -- devel@xxxxxxxxxxxxxxxxxxxxxxx
To unsubscribe send an email to devel-leave@xxxxxxxxxxxxxxxxxxxxxxx
Fedora Code of Conduct: https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: https://lists.fedoraproject.org/archives/list/devel@xxxxxxxxxxxxxxxxxxxxxxx
Do not reply to spam on the list, report it: https://pagure.io/fedora-infrastructure
