Fedora Account System
Red Hat Associate
Red Hat Customer
spec: https://fed500.fedorapeople.org/chinese-segmentation.spec srpm: https://fed500.fedorapeople.org/chinese-segmentation-0.^20240206git4a6b092-1.fc41.src.rpm description: Convert hanzi to pinyin and traditional chinese to simplified. fas: fed500 Reproducible: Always
srpm link in broken, fedorpeople -> fedorapeople
Edited it. Will likely need to update download link as Gitee requires authentication from most ip addresses, so may need to add a comment about that.
Thanks for the update, I was able to run a local fedora-review, I think there is an issue with the source, here is the first part of the output. Issues: ======= - Dist tag is present. - Sources used to build the package match the upstream source, as provided in the spec URL. Note: Upstream MD5sum check error, diff is in /sfs/fedora-review/review- chinese-segmentation/diff.txt See: https://docs.fedoraproject.org/en-US/packaging-guidelines/SourceURL/ can you review your src and the upstream's ?
Added a script to download from git spec: https://fed500.fedorapeople.org/chinese-segmentation.spec srpm: https://fed500.fedorapeople.org/chinese-segmentation-1.0.01-1.fc42.src.rpm
Copr build: https://copr.fedorainfracloud.org/coprs/build/9381360 (succeeded) Review template: https://download.copr.fedorainfracloud.org/results/@fedora-review/fedora-review-2358213-chinese-segmentation/fedora-rawhide-x86_64/09381360-chinese-segmentation/fedora-review/review.txt Please take a look if any issues were found. --- This comment was created by the fedora-review-service https://github.com/FrostyX/fedora-review-service If you want to trigger a new Copr build, add a comment containing new Spec and SRPM URLs or [fedora-review-service-build] string.
How are you running the fedora-review command? it is stumbling on source0,1 How are you getting this line ? License: GPL-3.0-or-later AND MIT AND NTP [!]: Package is licensed with an open-source compatible license and meets other legal requirements as defined in the legal section of Packaging Guidelines. It looks like this project has a few bundled projects with their license files stripped out, these are the ones i could easily find. This is the cppjieba/limonp is really this https://github.com/yanyiwu/limonp/tree/master/include/limonp And cppjiaba is really this https://github.com/yanyiwu/cppjieba A package can not strip other embedded license files.
This is an automatic check from review-stats script. This review request ticket hasn't been updated for some time, but it seems that the review is still being working out by you. If this is right, please respond to this comment clearing the NEEDINFO flag and try to reach out the submitter to proceed with the review. If you're not interested in reviewing this ticket anymore, please clear the fedora-review flag and reset the assignee, so that a new reviewer can take this ticket. Without any reply, this request will shortly be resetted.