[Bug 1840865] Review Request: lv2lint - LV2 turtle language checker

2020-12-12 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1840865 ycollet changed: What|Removed |Added Status|NEW |CLOSED Resolution|---

[Bug 1840865] Review Request: lv2lint - LV2 turtle language checker

2020-11-26 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1840865 ycollet changed: What|Removed |Added Status|CLOSED |NEW Resolution|NOTABUG

[Bug 1840865] Review Request: lv2lint - LV2 turtle language checker

2020-11-24 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1840865 ycollet changed: What|Removed |Added Status|ASSIGNED|CLOSED Resolution|---

[Bug 1840865] Review Request: lv2lint - LV2 turtle language checker

2020-07-09 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1840865 Petr Pisar changed: What|Removed |Added CC||ppi...@redhat.com

[Bug 1840865] Review Request: lv2lint - LV2 turtle language checker

2020-05-28 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1840865 --- Comment #24 from Vasiliy Glazov --- Yes, inform me. -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component

[Bug 1840865] Review Request: lv2lint - LV2 turtle language checker

2020-05-28 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1840865 --- Comment #23 from ycollet --- OK. I will certainly try to do a mix: submitting some new spec files and try to do some review. When I do a review, I add you to the review ? -- You are receiving this mail because: You are on the CC list

[Bug 1840865] Review Request: lv2lint - LV2 turtle language checker

2020-05-28 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1840865 --- Comment #22 from Vasiliy Glazov --- Because your spec was not good at start I need to make sure you understand most significant rules of Fedora packaging guidellines. https://docs.fedoraproject.org/en-US/packaging-guidelines/ First

[Bug 1840865] Review Request: lv2lint - LV2 turtle language checker

2020-05-28 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1840865 --- Comment #21 from ycollet --- I finished reading the wiki page related to sponsoring. What can I do so as you sponsor me ? -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes

[Bug 1840865] Review Request: lv2lint - LV2 turtle language checker

2020-05-27 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1840865 --- Comment #20 from Vasiliy Glazov --- I can be a sponsor. -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component

[Bug 1840865] Review Request: lv2lint - LV2 turtle language checker

2020-05-27 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1840865 --- Comment #19 from ycollet --- OK, thanks a lot for your review. I will now try to find a sponsor. -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and

[Bug 1840865] Review Request: lv2lint - LV2 turtle language checker

2020-05-27 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1840865 --- Comment #18 from Vasiliy Glazov --- Now you need to get sponsored. https://fedoraproject.org/wiki/Join_the_package_collection_maintainers#Get_Sponsored -- You are receiving this mail because: You are on the CC list for the bug. You are

[Bug 1840865] Review Request: lv2lint - LV2 turtle language checker

2020-05-27 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1840865 Vasiliy Glazov changed: What|Removed |Added Flags|fedora-review? |fedora-review+ --- Comment #17 from

[Bug 1840865] Review Request: lv2lint - LV2 turtle language checker

2020-05-27 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1840865 --- Comment #16 from ycollet --- Yes, you're right, c++ was not required. I cleaned up the spec file. The URL to the last version of the spec file:

[Bug 1840865] Review Request: lv2lint - LV2 turtle language checker

2020-05-27 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1840865 --- Comment #15 from Vasiliy Glazov --- It is really need gcc-c++ to build? I don't see it in build.log If not then remove it from BuildRequires. Source0: https://gitlab.com/drobilla/lv2lint/-/archive/%{version}/lv2lint-%{version}.tar.gz can

[Bug 1840865] Review Request: lv2lint - LV2 turtle language checker

2020-05-27 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1840865 Vasiliy Glazov changed: What|Removed |Added Status|NEW |ASSIGNED Flags|

[Bug 1840865] Review Request: lv2lint - LV2 turtle language checker

2020-05-27 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1840865 Vasiliy Glazov changed: What|Removed |Added Blocks||177841 (FE-NEEDSPONSOR)

[Bug 1840865] Review Request: lv2lint - LV2 turtle language checker

2020-05-27 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1840865 --- Comment #14 from ycollet --- I was waiting for the build to finish on COPR. The link to the lv2lint spec file on the COPR repo:

[Bug 1840865] Review Request: lv2lint - LV2 turtle language checker

2020-05-27 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1840865 --- Comment #13 from Vasiliy Glazov --- Now post new URLs. It needed to make review. -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component

[Bug 1840865] Review Request: lv2lint - LV2 turtle language checker

2020-05-27 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1840865 --- Comment #12 from ycollet --- Ah, sorry. I thought you where talking about the %buildroot in the build and install section. I commit a change ASAP. -- You are receiving this mail because: You are on the CC list for the bug. You are

[Bug 1840865] Review Request: lv2lint - LV2 turtle language checker

2020-05-27 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1840865 --- Comment #11 from Vasiliy Glazov --- Just use Spec URL: ... SRPM URL: ... As in first message. And you still not removed BuildRoot string from spec! -- You are receiving this mail because: You are on the CC list for the bug. You are

[Bug 1840865] Review Request: lv2lint - LV2 turtle language checker

2020-05-27 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1840865 --- Comment #10 from ycollet --- The link to the lv2lint spec file on the COPR repo: https://download.copr.fedorainfracloud.org/results/ycollet/linuxmao/fedora-32-x86_64/01414984-lv2lint/lv2lint.spec The link to the corresponding src rpm

[Bug 1840865] Review Request: lv2lint - LV2 turtle language checker

2020-05-27 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1840865 --- Comment #9 from Vasiliy Glazov --- Now please post direct links to updated Spec URL: and SRPM URL: -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and

[Bug 1840865] Review Request: lv2lint - LV2 turtle language checker

2020-05-27 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1840865 --- Comment #8 from ycollet --- OK, Thanks. I updated the build / install part. I removed %{set_build_flags} and used %meson %meson_build and %meson_install. I replaced the license by Artistic 2.0 and I fixed the %file part by using %{name}.

[Bug 1840865] Review Request: lv2lint - LV2 turtle language checker

2020-05-27 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1840865 --- Comment #7 from Vasiliy Glazov --- 8. Don't use glob * in %files section such way. Change paths to something like %{_bindir}/{name}* %{_mandir}/man1/%{name}.1* -- You are receiving this mail because: You are on the CC list for the bug.

[Bug 1840865] Review Request: lv2lint - LV2 turtle language checker

2020-05-27 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1840865 --- Comment #6 from Vasiliy Glazov --- 7. Change License: field to Artistic 2.0 Here you can find all allowed licenses and how they must be writed in License: field https://fedoraproject.org/wiki/Licensing:Main?rd=Licensing#SoftwareLicenses

[Bug 1840865] Review Request: lv2lint - LV2 turtle language checker

2020-05-27 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1840865 --- Comment #5 from Vasiliy Glazov --- No need patches here put it near your spec file. If you added patch then need %autosetup -p1 URL should be this https://raw.githubusercontent.com/ycollet/fedora-spec/master/lv2lint/lv2lint.spec You

[Bug 1840865] Review Request: lv2lint - LV2 turtle language checker

2020-05-27 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1840865 --- Comment #4 from ycollet --- I just fixed the spec file. I added a patch because of a duplicated symbol problem on Fedora 32. I don't understand the point 5 ? In the Review request, I must add the spec file and not give an URL to the

[Bug 1840865] Review Request: lv2lint - LV2 turtle language checker

2020-05-27 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1840865 --- Comment #3 from ycollet --- Created attachment 1692809 --> https://bugzilla.redhat.com/attachment.cgi?id=1692809=edit A requested patch -- You are receiving this mail because: You are on the CC list for the bug. You are always

[Bug 1840865] Review Request: lv2lint - LV2 turtle language checker

2020-05-27 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1840865 --- Comment #2 from ycollet --- Created attachment 1692808 --> https://bugzilla.redhat.com/attachment.cgi?id=1692808=edit The spec file -- You are receiving this mail because: You are on the CC list for the bug. You are always notified

[Bug 1840865] Review Request: lv2lint - LV2 turtle language checker

2020-05-27 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1840865 Vasiliy Glazov changed: What|Removed |Added CC||vasc...@gmail.com Doc