[Rpm-maint] [rpm-software-management/rpm] Add several new optional URL tags 2 (PR #2083)

Stanislav Brabec notifications at github.com
Wed Jun 15 13:29:29 UTC 2022


> I dunno about the usability problem with string arrays, these aren't something your Joe Average would be querying all day and night. If they need to have multiple values then I don't see why not use the technically correct solution instead of getting messy with delimiters which will inevitably need some form of escaping etc.

Then we should use:
```
    RPMTAG_UPSTREAMRELEASES	= 5101, /* s[] */
```
This will generate a parser accepting the array.

The other part of the patch is already ready for the array:
```
	SINGLE_TOKEN_ONLY;
	/* fallthrough */
    case RPMTAG_TRANSLATIONURL:
```

`SINGLE_TOKEN_ONLY` checks that no more that one token is used.

-- 
Reply to this email directly or view it on GitHub:
https://github.com/rpm-software-management/rpm/pull/2083#issuecomment-1156475155
You are receiving this because you are subscribed to this thread.

Message ID: <rpm-software-management/rpm/pull/2083/c1156475155 at github.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.rpm.org/pipermail/rpm-maint/attachments/20220615/514fe505/attachment.html>


More information about the Rpm-maint mailing list