[Bug 527306] Review Request: jericho-html - Jericho HTML Parser

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=527306

--- Comment #7 from Stanislav Ochotnicky <sochotni@xxxxxxxxxx> 2010-06-28 05:09:55 EDT ---
Ok, I am sorry for this, but I made a mistake when telling you to change group
to "Development/Documentation" it was supposed to be just "Documentation" (I
had to correct that exact problem so many times I remembered it as correct
solution). It is supposed to be just "Documentation". Also there is no group
"Development/Libraries/Java". Please look into /usr/share/doc/rpm-X.Y.Z/GROUPS
and pick one from there.

> - Don't need to copy library for tests  

You removed copying of jar file not for testing but for samples, and I believe
that sample will now stop working. Your change was:

61d61
< cp dist/%{name}-%{version}.jar samples/webapps/JerichoHTML/WEB-INF/lib

Now the JerichoHTML.war is missing jericho dependency. Why not just create
symlink of jericho jar file into samples/webapps/JerichoHTML/WEB-INF/lib and
install that without even creating the war file. I don't see any reason why we
should have both the war and the unpacked version.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.
_______________________________________________
package-review mailing list
package-review@xxxxxxxxxxxxxxxxxxxxxxx
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Index of Archives]     [Fedora Legacy]     [Fedora Desktop]     [Fedora SELinux]     [Yosemite News]     [KDE Users]     [Fedora Tools]