[Bug 973084] Review Request: htmlcleaner - HtmlCleaner is open-source HTML parser written in Java
https://bugzilla.redhat.com/show_bug.cgi?id=973084 Fedora Update System changed: What|Removed |Added Fixed In Version||htmlcleaner-2.2.1-2.fc19 Resolution|RAWHIDE |ERRATA --- Comment #29 from Fedora Update System --- htmlcleaner-2.2.1-2.fc19 has been pushed to the Fedora 19 stable repository. -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=208rtOYxK4&a=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 973084] Review Request: htmlcleaner - HtmlCleaner is open-source HTML parser written in Java
https://bugzilla.redhat.com/show_bug.cgi?id=973084 Björn Esser changed: What|Removed |Added Status|ON_QA |CLOSED Resolution|--- |RAWHIDE Last Closed||2013-07-19 09:22:17 -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=nL3ifpoT8E&a=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 973084] Review Request: htmlcleaner - HtmlCleaner is open-source HTML parser written in Java
https://bugzilla.redhat.com/show_bug.cgi?id=973084 Fedora Update System changed: What|Removed |Added Status|MODIFIED|ON_QA --- Comment #28 from Fedora Update System --- htmlcleaner-2.2.1-2.fc19 has been pushed to the Fedora 19 testing repository. -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=uBbPcSZmFr&a=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 973084] Review Request: htmlcleaner - HtmlCleaner is open-source HTML parser written in Java
https://bugzilla.redhat.com/show_bug.cgi?id=973084 --- Comment #27 from Fedora Update System --- htmlcleaner-2.2.1-2.fc19 has been submitted as an update for Fedora 19. https://admin.fedoraproject.org/updates/htmlcleaner-2.2.1-2.fc19 -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=z7guycHNON&a=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 973084] Review Request: htmlcleaner - HtmlCleaner is open-source HTML parser written in Java
https://bugzilla.redhat.com/show_bug.cgi?id=973084 Fedora Update System changed: What|Removed |Added Status|ASSIGNED|MODIFIED -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=aJIrNoFtC7&a=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 973084] Review Request: htmlcleaner - HtmlCleaner is open-source HTML parser written in Java
https://bugzilla.redhat.com/show_bug.cgi?id=973084 --- Comment #26 from Björn Esser --- (In reply to Marcin.Dulak from comment #25) > OK. i think i have those koji builds, what to do next, 'comps'? So all went fine so far and you now have ready-build pkgs for rawhide and F19. For rawhide your pkg gets imported automaticly after it's been built. If you're not familiar with `vim` as editor you probably want to use another, e.g. `nano`: `export EDITOR="$my_fav"` Replace $my_fav with your prefered editor. You can make this system-default by invoking `echo 'export EDITOR="$my_fav"' > /etc/profile.d/preferred_editor.sh` as root. For the F19 branch you need to tell "bodhi" to push it to the repo: http://fedoraproject.org/wiki/Join_the_package_collection_maintainers#Submit_Package_as_Update_in_Bodhi `fedpkg switch-branch f19 ; fedpkg update` After invoking the commands your editor opens giving you a template to complete: [ htmlcleaner-2.2.1-2.fc19 ] # bugfix, security, enhancement, newpackage (required) type= <--- here goes the type of the update, in your case it's newpackage. enhancement is meant for new upstream release. bugfix and security are self explaining, I think. # testing, stable request=testing<--- keep the default here, directly pushing to stable repo is meant for really urgent security-fixes. (type=security) # Bug numbers: 1234,9876 bugs=973084,11 <--- just delete the `,11` here, bodhi will close your review-bug automatically then. See below. # Description of your update notes= <--- Here is where you give an explanation of your update. On newpkgs I'd propose %description, on enhancements or bugfixes %changelog. All in a row without manual line-breaks. # Enable request automation based on the stable/unstable karma thresholds autokarma=True <--- you can keep the defaults here. stable_karma=3 unstable_karma=-3 # Automatically close bugs when this marked as stable close_bugs=True<--- See above, bug numbers. # Suggest that users restart after update suggest_reboot=False <--- mostly needed for kernel-updates or some daemons. After you saved the modded template and close the editor your update-request will be send to bodhi. You'll need your FAS-password during progress. You can push your package to stable after 7 days in testing (bodhi will inform you by mail) or it gets auto-pushed when needed karma was reached (other packager-people may test your package and give +1/-1-votes). Adding libs to comps should be avoided, see: https://fedoraproject.org/wiki/How_to_use_and_edit_comps.xml_for_package_groups?rd=PackageMaintainers/CompsXml#When_to_Edit_comps | Libraries should not be included - they will be pulled in via dependencies You surely want to add your package to Upstream Release Monitoring: http://fedoraproject.org/wiki/Join_the_package_collection_maintainers#Watch_for_updates https://fedoraproject.org/wiki/Upstream_release_monitoring?rd=Upstream_Release_Monitoring If your package is added there, you'll get a new bug opened on bugzilla informing you by mail, everytime there's a new upstream version released. -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=Iml4fY7alE&a=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 973084] Review Request: htmlcleaner - HtmlCleaner is open-source HTML parser written in Java
https://bugzilla.redhat.com/show_bug.cgi?id=973084 --- Comment #25 from marcin.du...@gmail.com --- (In reply to Björn Esser from comment #24) > (In reply to Marcin.Dulak from comment #23) > > see already few interesting candidates for review there > > So you may want to add yourself to CC of the bugs tracking changes: > > * go to the bug > * check the "Add me to CC list" > * --> "Save changes" > > > So, limb has setup the git-repo for importing your package. At this point > you want to make sure having > > * installed the pkg from "fedora-packager"-group: > `yum groups install fedora-packager` > > * have imported your rsa pub-key into your account: > https://admin.fedoraproject.org/accounts/user/edit/marcindulak > > * have run `fedora-packager-setup` and imported the generated > certificate into your browser, so you can login into koji: > https://koji.fedoraproject.org/koji/index > > After you have completed these steps. You should follow the instructions > shown in the wiki: > http://fedoraproject.org/wiki/ > Join_the_package_collection_maintainers#Check_out_the_module > > Some overview for advanced use of `fedpkg` is given here: > http://fedoraproject.org/wiki/Using_git_FAQ_for_package_maintainers > > If you have any questions or problems feel free to ask me! OK. i think i have those koji builds, what to do next, 'comps'? -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=xiSgLcpQS5&a=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 973084] Review Request: htmlcleaner - HtmlCleaner is open-source HTML parser written in Java
https://bugzilla.redhat.com/show_bug.cgi?id=973084 --- Comment #24 from Björn Esser --- (In reply to Marcin.Dulak from comment #23) > see already few interesting candidates for review there So you may want to add yourself to CC of the bugs tracking changes: * go to the bug * check the "Add me to CC list" * --> "Save changes" So, limb has setup the git-repo for importing your package. At this point you want to make sure having * installed the pkg from "fedora-packager"-group: `yum groups install fedora-packager` * have imported your rsa pub-key into your account: https://admin.fedoraproject.org/accounts/user/edit/marcindulak * have run `fedora-packager-setup` and imported the generated certificate into your browser, so you can login into koji: https://koji.fedoraproject.org/koji/index After you have completed these steps. You should follow the instructions shown in the wiki: http://fedoraproject.org/wiki/Join_the_package_collection_maintainers#Check_out_the_module Some overview for advanced use of `fedpkg` is given here: http://fedoraproject.org/wiki/Using_git_FAQ_for_package_maintainers If you have any questions or problems feel free to ask me! -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=MgzwC7gOGY&a=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 973084] Review Request: htmlcleaner - HtmlCleaner is open-source HTML parser written in Java
https://bugzilla.redhat.com/show_bug.cgi?id=973084 --- Comment #23 from marcin.du...@gmail.com --- (In reply to Björn Esser from comment #21) > (In reply to Marcin.Dulak from comment #20) > > yes, i was surprised i can do that myself. Thanks for fixing that! > > You're welcome! > > Yes, now you are a packager, you can, but mustn't do on your own packages. > :) You can take official reviews for other packagers, too. The comments > you made in [1] look really good and taking this to full review && approval > sounds like a good start. OK > > You may want to install the `fedora-review` package providing all basic > stuff needed. This is a half-automated tool checking some stuff, but may > report false-positives or miss some. It doesn't check everything, so some > manual work will be needed either. already using fedora-review, it helps a lot. > > An introduction about the review process you can find here. [2] And here's > [3] a list with packages waiting for review; those with green bg need review > by a packager-sponsor. > > [1] bz #961180 > [2] https://fedoraproject.org/wiki/Packaging/ReviewGuidelines > [3] http://fedoraproject.org/PackageReviewStatus/NEW.html see already few interesting candidates for review there -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=V5Qv2Fk6Fr&a=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 973084] Review Request: htmlcleaner - HtmlCleaner is open-source HTML parser written in Java
https://bugzilla.redhat.com/show_bug.cgi?id=973084 --- Comment #22 from Jon Ciesla --- Git done (by process-git-requests). -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=P0j9F4ewXp&a=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 973084] Review Request: htmlcleaner - HtmlCleaner is open-source HTML parser written in Java
https://bugzilla.redhat.com/show_bug.cgi?id=973084 Jon Ciesla changed: What|Removed |Added Flags|fedora-cvs? | Flags||fedora-cvs+ -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=dnMBa6QqiR&a=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 973084] Review Request: htmlcleaner - HtmlCleaner is open-source HTML parser written in Java
https://bugzilla.redhat.com/show_bug.cgi?id=973084 --- Comment #21 from Björn Esser --- (In reply to Marcin.Dulak from comment #20) > yes, i was surprised i can do that myself. Thanks for fixing that! You're welcome! Yes, now you are a packager, you can, but mustn't do on your own packages. :) You can take official reviews for other packagers, too. The comments you made in [1] look really good and taking this to full review && approval sounds like a good start. You may want to install the `fedora-review` package providing all basic stuff needed. This is a half-automated tool checking some stuff, but may report false-positives or miss some. It doesn't check everything, so some manual work will be needed either. An introduction about the review process you can find here. [2] And here's [3] a list with packages waiting for review; those with green bg need review by a packager-sponsor. [1] bz #961180 [2] https://fedoraproject.org/wiki/Packaging/ReviewGuidelines [3] http://fedoraproject.org/PackageReviewStatus/NEW.html -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=6oDfCL8ih7&a=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 973084] Review Request: htmlcleaner - HtmlCleaner is open-source HTML parser written in Java
https://bugzilla.redhat.com/show_bug.cgi?id=973084 --- Comment #20 from marcin.du...@gmail.com --- (In reply to Björn Esser from comment #19) > Please be careful with the flags. :) I fixed them. If you set > fedora-review to (+) for others it looks like you granted review yourself. > But never mind this happend to me on my first packages as well. ;) yes, i was surprised i can do that myself. Thanks for fixing that! -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=GtsKlT4Jw7&a=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 973084] Review Request: htmlcleaner - HtmlCleaner is open-source HTML parser written in Java
https://bugzilla.redhat.com/show_bug.cgi?id=973084 Björn Esser changed: What|Removed |Added Flags|fedora-review+ | Flags||fedora-review? --- Comment #19 from Björn Esser --- Please be careful with the flags. :) I fixed them. If you set fedora-review to (+) for others it looks like you granted review yourself. But never mind this happend to me on my first packages as well. ;) -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=kI34qTmjm2&a=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 973084] Review Request: htmlcleaner - HtmlCleaner is open-source HTML parser written in Java
https://bugzilla.redhat.com/show_bug.cgi?id=973084 Björn Esser changed: What|Removed |Added Flags|fedora-review? | Flags||fedora-review+ -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=657zwy60wi&a=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 973084] Review Request: htmlcleaner - HtmlCleaner is open-source HTML parser written in Java
https://bugzilla.redhat.com/show_bug.cgi?id=973084 marcin.du...@gmail.com changed: What|Removed |Added Flags|fedora-review? | Flags||fedora-review+ Flags||fedora-cvs? -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=fnPvdnKmeZ&a=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 973084] Review Request: htmlcleaner - HtmlCleaner is open-source HTML parser written in Java
https://bugzilla.redhat.com/show_bug.cgi?id=973084 marcin.du...@gmail.com changed: What|Removed |Added Flags|fedora-review+ | Flags||fedora-review? --- Comment #18 from marcin.du...@gmail.com --- New Package SCM Request === Package Name: htmlcleaner Short Description: HTML parser written in Java Owners: marcindulak besser82 Branches: f19 InitialCC: java-sig -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=189LLDYynA&a=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 973084] Review Request: htmlcleaner - HtmlCleaner is open-source HTML parser written in Java
https://bugzilla.redhat.com/show_bug.cgi?id=973084 --- Comment #17 from Björn Esser --- Just forgot: You're doing a nice work on bz #961180 Please add me `besser82` as co-maintainer for SCM and set `InitialCC: java-sig`, please. So I can help you, if there's something related to packaging and java-oriented fedora-people get aware there's a new java-package around. -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=E6Etor2URi&a=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 973084] Review Request: htmlcleaner - HtmlCleaner is open-source HTML parser written in Java
https://bugzilla.redhat.com/show_bug.cgi?id=973084 Björn Esser changed: What|Removed |Added Status|NEW |ASSIGNED Blocks|177841 (FE-NEEDSPONSOR) | Assignee|nob...@fedoraproject.org|bjoern.es...@gmail.com Flags||fedora-review+ --- Comment #16 from Björn Esser --- (In reply to Marcin.Dulak from comment #15) > > 's!BSD$!& with advertising!' > > the /usr/share/doc/htmlcleaner-2.2.1/licence.txt file > looks to me closer to > https://fedoraproject.org/wiki/Licensing:BSD?rd=Licensing/BSD#3ClauseBSD > It's right that 13 files do not contain license header, but why > changing the license to "BSD with advertising"? You're right of course. The part about: | Neither the name of the nor the names of its contributors | may be used to endorse or promote products derived from this software | without specific prior written permission got me on the wrong track... ;) Note: You possibly want to rename `licence.txt` to proper spelling, but that's up to you deciding about: %prep ln -f LICENSE licence.txt %doc LICENSE Your package is fine to be imported to SCM now! APPROVED! -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=8ndCgLcbmH&a=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 973084] Review Request: htmlcleaner - HtmlCleaner is open-source HTML parser written in Java
https://bugzilla.redhat.com/show_bug.cgi?id=973084
--- Comment #15 from marcin.du...@gmail.com ---
(In reply to Björn Esser from comment #14)
> Package is fine now, except for License-Tag in spec:
>
> 's!BSD$!& with advertising!'
the /usr/share/doc/htmlcleaner-2.2.1/licence.txt file
looks to me closer to
https://fedoraproject.org/wiki/Licensing:BSD?rd=Licensing/BSD#3ClauseBSD
The modification are:
- range of instead of
- a disclaimer at the bottom of the file saying:
"You can contact Vladimir Nikic by sending e-mail to
nikic_vladi...@yahoo.com. Please include the word "HtmlCleaner" in the
subject line."
It's right that 13 files do not contain license header, but why
changing the license to "BSD with advertising"?
> #
>
> Package Review
> ==
>
> Legend:
> [x] = Pass
> [!] = Fail
> [-] = Not applicable
> [?] = Not evaluated
> [ ] = Manual review needed
>
>
>
> = MUST items =
>
> Generic:
> [x]: Package is licensed with an open-source compatible license and meets
> other legal requirements as defined in the legal section of Packaging
> Guidelines.
> [x]: Package contains no bundled libraries without FPC exception.
> [x]: Changelog in prescribed format.
> [x]: Sources contain only permissible code or content.
> [-]: Package contains desktop file if it is a GUI application.
> [x]: Development files must be in a -devel package
> [x]: Package requires other packages for directories it uses.
> [x]: Package uses nothing in %doc for runtime.
> [x]: Package is not known to require ExcludeArch.
> [x]: Fully versioned dependency in subpackages, if present.
> Note: No Requires: %{name}%{?_isa} = %{version}-%{release} in
> htmlcleaner-javadoc
>
> ---> false positive, see comment #11
>
> [x]: Package complies to the Packaging Guidelines
> [!]: License field in the package spec file matches the actual license.
> Note: Checking patched sources after %prep for licenses. Licenses found:
> "BSD (3 clause)", "Unknown or generated". 13 files have unknown license.
> Detailed output of licensecheck in
> /home/bjoern.esser/fedora/review/973084-htmlcleaner/licensecheck.txt
>
> ---> see above: 's!BSD!& with advertising!'
>
> [x]: License file installed when any subpackage combination is installed.
> [x]: Package consistently uses macro is (instead of hard-coded directory
> names).
> [x]: Package is named according to the Package Naming Guidelines.
> [x]: Package does not generate any conflict.
> [x]: Package obeys FHS, except libexecdir and /usr/target.
> [x]: If the package is a rename of another package, proper Obsoletes and
> Provides are present.
> [x]: Package must own all directories that it creates.
> [x]: Package does not own files or directories owned by other packages.
> [x]: Requires correct, justified where necessary.
> [x]: Spec file is legible and written in American English.
> [-]: Package contains systemd file(s) if in need.
> [-]: Large documentation must go in a -doc subpackage.
> Note: Documentation size is 10240 bytes in 1 files.
> [x]: All build dependencies are listed in BuildRequires, except for any that
> are listed in the exceptions section of Packaging Guidelines.
> [x]: Package does not run rm -rf %{buildroot} (or $RPM_BUILD_ROOT) at the
> beginning of %install.
> [x]: Each %files section contains %defattr if rpm < 4.4
> [x]: Macros in Summary, %description expandable at SRPM build time.
> [x]: Package does not contain duplicates in %files.
> [x]: Permissions on files are set properly.
> [x]: If (and only if) the source package includes the text of the license(s)
> in its own file, then that file, containing the text of the license(s)
> for the package is included in %doc.
> [x]: Package use %makeinstall only when make install' ' DESTDIR=... doesn't
> work.
> [x]: Package is named using only allowed ASCII characters.
> [x]: Package do not use a name that already exist
> [x]: Package is not relocatable.
> [x]: Sources used to build the package match the upstream source, as provided
> in the spec URL.
> [x]: Spec file name must match the spec package %{name}, in the format
> %{name}.spec.
> [x]: File names are valid UTF-8.
> [x]: Packages must not store files under /srv, /opt or /usr/local
> [x]: Package successfully compiles and builds into binary rpms on at least
> one
> supported primary architecture.
> [x]: Package installs properly.
> [x]: Rpmlint is run on all rpms the build produces.
> Note: No rpmlint messages.
>
> Java:
> [x]: Packages have proper BuildRequires/Requires on jpackage-utils
> Note: Maven packages do not need to (Build)Require jpackage-utils. It is
> pulled in by maven-local
> [x]: Fully versioned dependency in subpackages, if present.
> [x]: Javadoc documentation files are generated and included in -javadoc
> subpackage
> [x]: Javadoc subpackages should not have Requires: jpackage-utils
> [x]: Javadocs are placed in %
[Bug 973084] Review Request: htmlcleaner - HtmlCleaner is open-source HTML parser written in Java
https://bugzilla.redhat.com/show_bug.cgi?id=973084
--- Comment #14 from Björn Esser ---
Package is fine now, except for License-Tag in spec:
's!BSD$!& with advertising!'
#
Package Review
==
Legend:
[x] = Pass
[!] = Fail
[-] = Not applicable
[?] = Not evaluated
[ ] = Manual review needed
= MUST items =
Generic:
[x]: Package is licensed with an open-source compatible license and meets
other legal requirements as defined in the legal section of Packaging
Guidelines.
[x]: Package contains no bundled libraries without FPC exception.
[x]: Changelog in prescribed format.
[x]: Sources contain only permissible code or content.
[-]: Package contains desktop file if it is a GUI application.
[x]: Development files must be in a -devel package
[x]: Package requires other packages for directories it uses.
[x]: Package uses nothing in %doc for runtime.
[x]: Package is not known to require ExcludeArch.
[x]: Fully versioned dependency in subpackages, if present.
Note: No Requires: %{name}%{?_isa} = %{version}-%{release} in
htmlcleaner-javadoc
---> false positive, see comment #11
[x]: Package complies to the Packaging Guidelines
[!]: License field in the package spec file matches the actual license.
Note: Checking patched sources after %prep for licenses. Licenses found:
"BSD (3 clause)", "Unknown or generated". 13 files have unknown license.
Detailed output of licensecheck in
/home/bjoern.esser/fedora/review/973084-htmlcleaner/licensecheck.txt
---> see above: 's!BSD!& with advertising!'
[x]: License file installed when any subpackage combination is installed.
[x]: Package consistently uses macro is (instead of hard-coded directory
names).
[x]: Package is named according to the Package Naming Guidelines.
[x]: Package does not generate any conflict.
[x]: Package obeys FHS, except libexecdir and /usr/target.
[x]: If the package is a rename of another package, proper Obsoletes and
Provides are present.
[x]: Package must own all directories that it creates.
[x]: Package does not own files or directories owned by other packages.
[x]: Requires correct, justified where necessary.
[x]: Spec file is legible and written in American English.
[-]: Package contains systemd file(s) if in need.
[-]: Large documentation must go in a -doc subpackage.
Note: Documentation size is 10240 bytes in 1 files.
[x]: All build dependencies are listed in BuildRequires, except for any that
are listed in the exceptions section of Packaging Guidelines.
[x]: Package does not run rm -rf %{buildroot} (or $RPM_BUILD_ROOT) at the
beginning of %install.
[x]: Each %files section contains %defattr if rpm < 4.4
[x]: Macros in Summary, %description expandable at SRPM build time.
[x]: Package does not contain duplicates in %files.
[x]: Permissions on files are set properly.
[x]: If (and only if) the source package includes the text of the license(s)
in its own file, then that file, containing the text of the license(s)
for the package is included in %doc.
[x]: Package use %makeinstall only when make install' ' DESTDIR=... doesn't
work.
[x]: Package is named using only allowed ASCII characters.
[x]: Package do not use a name that already exist
[x]: Package is not relocatable.
[x]: Sources used to build the package match the upstream source, as provided
in the spec URL.
[x]: Spec file name must match the spec package %{name}, in the format
%{name}.spec.
[x]: File names are valid UTF-8.
[x]: Packages must not store files under /srv, /opt or /usr/local
[x]: Package successfully compiles and builds into binary rpms on at least one
supported primary architecture.
[x]: Package installs properly.
[x]: Rpmlint is run on all rpms the build produces.
Note: No rpmlint messages.
Java:
[x]: Packages have proper BuildRequires/Requires on jpackage-utils
Note: Maven packages do not need to (Build)Require jpackage-utils. It is
pulled in by maven-local
[x]: Fully versioned dependency in subpackages, if present.
[x]: Javadoc documentation files are generated and included in -javadoc
subpackage
[x]: Javadoc subpackages should not have Requires: jpackage-utils
[x]: Javadocs are placed in %{_javadocdir}/%{name} (no -%{version} symlink)
[x]: Bundled jar/class files should be removed before build
Maven:
[x]: If package contains pom.xml files install it (including depmaps) even
when building with ant
[x]: Pom files have correct Maven mapping
[x]: Maven packages should use new style packaging
[x]: Old add_to_maven_depmap macro is not being used
[x]: Packages DOES NOT have Requires(post) and Requires(postun) on jpackage-
utils for %update_maven_depmap macro
[x]: Package DOES NOT use %update_maven_depmap in %post/%postun
[x]: Packages use %{_mavenpomdir} instead of %{_datadir}/maven2/poms
= SHOULD items =
Generic:
[-]: If the source package does not include license text(s) as a separate file
from upstream, th
[Bug 973084] Review Request: htmlcleaner - HtmlCleaner is open-source HTML parser written in Java
https://bugzilla.redhat.com/show_bug.cgi?id=973084 --- Comment #13 from Mikolaj Izdebski --- (In reply to Marcin.Dulak from comment #12) > By the way what's the correct way of running rpmlint, when i do: > rpmlint htmlcleaner.spec > i don't get any errors/warnings. You should run rpmlint on SRPM instead of spec file. -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=0e7dITJFGd&a=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 973084] Review Request: htmlcleaner - HtmlCleaner is open-source HTML parser written in Java
https://bugzilla.redhat.com/show_bug.cgi?id=973084 --- Comment #12 from marcin.du...@gmail.com --- http://marcindulak.fedorapeople.org/packages/htmlcleaner/v02/htmlcleaner.spec http://marcindulak.fedorapeople.org/packages/htmlcleaner/v02/htmlcleaner-2.2.1-2.fc20.src.rpm Issues from bug #973084 comment #11 should be fixed. By the way what's the correct way of running rpmlint, when i do: rpmlint htmlcleaner.spec i don't get any errors/warnings. -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=6rwnGejyUc&a=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 973084] Review Request: htmlcleaner - HtmlCleaner is open-source HTML parser written in Java
https://bugzilla.redhat.com/show_bug.cgi?id=973084
--- Comment #11 from Björn Esser ---
Package has some issues. I'll collect things found by others in previous
comments and those I found in my report. build.log shows nothing to worry.
#
Package Review
==
Legend:
[x] = Pass
[!] = Fail
[-] = Not applicable
[?] = Not evaluated
[ ] = Manual review needed
Issues:
===
- Maven packages should use new style packaging
Note: If possible update your package to latest guidelines
See: https://fedoraproject.org/wiki/Packaging:Java#Apache_Maven
---> If you're really in the need having/using this in F18 you can keep
"old-style" until F18 goes EOL, but you should add some comment about
changing it to new style after F18 is EOL'ed.
Otherwise you should change it to new-style immediatly and only build
your package for F19+, only.
- All build dependencies are listed in BuildRequires, except for any that are
listed in the exceptions section of Packaging Guidelines.
Note: These BR are not needed: unzip
See: http://fedoraproject.org/wiki/Packaging/Guidelines#Exceptions_2
---> you can safely remove this BR, it's automaticly avail on all build-envs
- Packages have proper BuildRequires/Requires on jpackage-utils
Note: Maven packages do not need to (Build)Require jpackage-utils. It is
pulled in by maven-local
See: https://fedoraproject.org/wiki/Packaging:Java
---> there's no need for BR: java-devel jpackage-utils,
they are installed in build-env by dependency
No need for conditional on BRs, too. In this case it would be needed
for Fedora <= 17, only. maven-local is avail on F18+
- Javadoc subpackages should not have Requires: jpackage-utils
Note: jpackage-utils requires are automatically generated by the buildsystem
See: https://fedoraproject.org/wiki/Packaging:Java
---> please remove Requires: jpackage-utils,
it's picked-up by autorequires
- rpmlint complains (as seen below, too):
htmlcleaner.spec:7: W: mixed-use-of-spaces-and-tabs (spaces: line 7, tab:
line 1)
---> use tabs OR spaces for spacing columns, don't mix. see: comment #9
htmlcleaner.noarch: W: name-repeated-in-summary C HtmlCleaner
---> proposed solution: `Summary:HTML parser written in Java` should be
fine
htmlcleaner.noarch: E: description-line-too-long C HtmlCleaner is
open-source HTML parser written in Java. ...
---> please split lines @80 char max.
htmlcleaner.noarch: W: wrong-file-end-of-line-encoding
/usr/share/doc/htmlcleaner-2.2.1/licence.txt
---> this can be easiely fixed:
sed -i -e 's!\r!!g' licence.txt
see:
https://fedoraproject.org/wiki/Common_Rpmlint_issues#wrong-file-end-of-line-encoding
The rest of complains is just repeated from the above.
3 packages and 1 specfiles checked; 2 errors, 5 warnings.
= MUST items =
Generic:
[x]: Package is licensed with an open-source compatible license and meets
other legal requirements as defined in the legal section of Packaging
Guidelines.
[x]: Package contains no bundled libraries without FPC exception.
[x]: Changelog in prescribed format.
[x]: Sources contain only permissible code or content.
[-]: Package contains desktop file if it is a GUI application.
[-]: Development files must be in a -devel package
[x]: Package requires other packages for directories it uses.
[x]: Package uses nothing in %doc for runtime.
[x]: Package is not known to require ExcludeArch.
[x]: Fully versioned dependency in subpackages, if present.
Note: No Requires: %{name}%{?_isa} = %{version}-%{release} in
htmlcleaner-javadoc
---> false positive: docs-pkgs should not require the "binary" app.
btw. noarch-pkgs mustn't have %{?_isa}-macro on requires.
[!]: Package complies to the Packaging Guidelines
---> issues are found
[x]: License field in the package spec file matches the actual license.
Note: Checking patched sources after %prep for licenses. Licenses found:
"BSD (3 clause)", "Unknown or generated". 13 files have unknown license.
Detailed output of licensecheck in
/home/bjoern.esser/fedora/review/973084-htmlcleaner/licensecheck.txt
---> License-Tag should be: "BSD with advertising", since license
is 3-clause-BSD in fact.
[!]: License file installed when any subpackage combination is installed.
---> javadoc-pkg should include license-file, too. This is OK according
to guidelines, see:
https://fedoraproject.org/wiki/Packaging/Guidelines#Duplicate_Files
https://fedoraproject.org/wiki/Packaging:LicensingGuidelines#Subpackage_Licensing
[x]: Package consistently uses macro is (instead of hard-coded directory
names).
[x]: Package is named according to the Package Naming Guidelines.
[x]: Package does not generate any conflict.
[x]: Package obeys FHS, except libexecdir and /usr/target.
[-]: If the package is a rename of another p
[Bug 973084] Review Request: htmlcleaner - HtmlCleaner is open-source HTML parser written in Java
https://bugzilla.redhat.com/show_bug.cgi?id=973084 --- Comment #10 from Mikolaj Izdebski --- (In reply to Marcin.Dulak from comment #4) > The https://fedorahosted.org/packager-sponsors/ticket/69 step > is blocking me for almost a day now You may have better chances finding a reviewer quickly if you block Java SIG tracker bug #652183 (FE-JAVASIG), post on java-devel mailing list or join #fedora-java on IRC and ask. -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=iwSh0GRNsI&a=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 973084] Review Request: htmlcleaner - HtmlCleaner is open-source HTML parser written in Java
https://bugzilla.redhat.com/show_bug.cgi?id=973084 Christopher Meng changed: What|Removed |Added CC||cicku...@gmail.com --- Comment #9 from Christopher Meng --- Besides these from Mikolaj Izdebski, you also need to: Cleanup your spec, DO NOT mix tab and space. I think Fedora 17 is near EOL, so you can ignore Fedora 18-. -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=mOhLsI0FEv&a=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 973084] Review Request: htmlcleaner - HtmlCleaner is open-source HTML parser written in Java
https://bugzilla.redhat.com/show_bug.cgi?id=973084
Mikolaj Izdebski changed:
What|Removed |Added
CC||mizde...@redhat.com
--- Comment #8 from Mikolaj Izdebski ---
Disclaimer: I am not a sponsor of packager group so I can't do a formal review.
I am adding comments as member of Java SIG and maintainer of Maven in Fedora.
1. You can use -f argument to %files macro. For example, instead of:
%files
%{_javadir}/%{name}.jar
%{_mavendepmapfragdir}/%{name}
%{_mavenpomdir}/JPP-%{name}.pom
you can simply write
%files -f .mfiles
2. The call to %add_maven_depmap can be simplified, just remove both arguments.
3. "rm -rf $RPM_BUILD_ROOT" is not needed and should be removed. (It would be
needed for EPEL 5, but there is no Maven in EPEL.)
4. Group tags are not used in Fedora and can be removed.
5. Package descriptions should be wrapped up at 80 columns. (Other lines can
be longer, but should also be kept short if possible.)
6. Instead of using unzip you can use jar utility. It is included with
java-devel, so you wouldn't need to add BR on unzip. Basically remove that BR
and s/unzip/jar xf/ in %prep.
7. To patch pom.xml files you are recommended to use %pom_* macros. Instead of
adding patch a single line in prep would suffice:
%pom_remove_plugin :maven-gpg-plugin
--
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug
https://bugzilla.redhat.com/token.cgi?t=ieNfgSCx8a&a=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 973084] Review Request: htmlcleaner - HtmlCleaner is open-source HTML parser written in Java
https://bugzilla.redhat.com/show_bug.cgi?id=973084 --- Comment #7 from marcin.du...@gmail.com --- Spec URL: http://marcindulak.fedorapeople.org/packages/htmlcleaner/v01/htmlcleaner.spec SRPM URL: http://marcindulak.fedorapeople.org/packages/htmlcleaner/v01/htmlcleaner-2.2.1-1.fc20.src.rpm -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=YTJWJzclVe&a=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 973084] Review Request: htmlcleaner - HtmlCleaner is open-source HTML parser written in Java
https://bugzilla.redhat.com/show_bug.cgi?id=973084 --- Comment #6 from marcin.du...@gmail.com --- (In reply to Michael Schwendt from comment #5) > Please show a little bit of patience. It's a ticketing system with human > volunteers working on those tasks. OK -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=H0pIENQuW8&a=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 973084] Review Request: htmlcleaner - HtmlCleaner is open-source HTML parser written in Java
https://bugzilla.redhat.com/show_bug.cgi?id=973084 --- Comment #5 from Michael Schwendt --- Please show a little bit of patience. It's a ticketing system with human volunteers working on those tasks. -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=CZlKFEIwzC&a=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 973084] Review Request: htmlcleaner - HtmlCleaner is open-source HTML parser written in Java
https://bugzilla.redhat.com/show_bug.cgi?id=973084 --- Comment #4 from marcin.du...@gmail.com --- The https://fedorahosted.org/packager-sponsors/ticket/69 step is blocking me for almost a day now -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=omqn5GKnI9&a=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 973084] Review Request: htmlcleaner - HtmlCleaner is open-source HTML parser written in Java
https://bugzilla.redhat.com/show_bug.cgi?id=973084 --- Comment #3 from Björn Esser --- Thanks! Don't forget to sign "Contributor Agreement" inside your FAS-account and to request webspace for direct file-linking (you need to login with your FAS-credentials): https://fedorahosted.org/packager-sponsors/newticket Just select "initial package hosting request" and refer to this bug in your ticket. Upload your spec and srpm to your fedorapeople webspace when it's ready and update the links, please. -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=OntrxRG5jC&a=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 973084] Review Request: htmlcleaner - HtmlCleaner is open-source HTML parser written in Java
https://bugzilla.redhat.com/show_bug.cgi?id=973084 --- Comment #2 from marcin.du...@gmail.com --- Fedora Account System Username: marcindulak -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=t7DKeiyqTJ&a=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 973084] Review Request: htmlcleaner - HtmlCleaner is open-source HTML parser written in Java
https://bugzilla.redhat.com/show_bug.cgi?id=973084 Björn Esser changed: What|Removed |Added CC||bjoern.es...@gmail.com Blocks||177841 (FE-NEEDSPONSOR), ||894413 --- Comment #1 from Björn Esser --- Sign-up on Fedora Account System, if you haven't done already and provide your FAS-username, please. Added Blocks: FE-NEEDSPONSOR, since look-up for email-address in packager-group gave me no result. -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=vUGES8NqhI&a=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
