[Bug 655601] Review Request: jing-trang - Schema validation and conversion based on RELAX NG

2011-01-04 Thread bugzilla
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=655601

Paul Howarth p...@city-fan.org changed:

   What|Removed |Added

 CC||p...@city-fan.org

--- Comment #20 from Paul Howarth p...@city-fan.org 2011-01-04 04:22:58 EST 
---
How come there's no build of this in Rawhide?

-- 
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@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 655601] Review Request: jing-trang - Schema validation and conversion based on RELAX NG

2011-01-04 Thread bugzilla
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=655601

--- Comment #21 from Ville Skyttä ville.sky...@iki.fi 2011-01-04 12:37:14 EST 
---
I thought that the rawhide compose would pull it automatically in from previous
release updates like it tends to do if a package hasn't been built for rawhide
and there's a newer version in previous released updates, but perhaps it
doesn't work the same way for new packages.  Anyway, it's built for rawhide now
as well.

-- 
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@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 655601] Review Request: jing-trang - Schema validation and conversion based on RELAX NG

2011-01-04 Thread bugzilla
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=655601

--- Comment #22 from Paul Howarth p...@city-fan.org 2011-01-04 12:42:26 EST 
---
I think inheritance was deliberately broken between F14 and Rawhide for
something related to deltarpm creation and the new xz file format in Rawhide.

-- 
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@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 655601] Review Request: jing-trang - Schema validation and conversion based on RELAX NG

2011-01-04 Thread bugzilla
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=655601

--- Comment #23 from Ville Skyttä ville.sky...@iki.fi 2011-01-04 13:39:11 EST 
---
I see.  I've just posted to the devel list asking for clarification.

-- 
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@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 655601] Review Request: jing-trang - Schema validation and conversion based on RELAX NG

2010-12-13 Thread bugzilla
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=655601

Alexander Kurtakov akurt...@redhat.com changed:

   What|Removed |Added

 Blocks|652183(FE-JAVASIG)  |

-- 
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@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 655601] Review Request: jing-trang - Schema validation and conversion based on RELAX NG

2010-12-12 Thread bugzilla
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=655601

--- Comment #19 from Fedora Update System upda...@fedoraproject.org 
2010-12-12 15:03:08 EST ---
jing-trang-2009-7.fc14 has been pushed to the Fedora 14 stable repository. 
If problems still persist, please make note of it in this bug report.

-- 
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@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 655601] Review Request: jing-trang - Schema validation and conversion based on RELAX NG

2010-12-04 Thread bugzilla
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=655601

Fedora Update System upda...@fedoraproject.org changed:

   What|Removed |Added

 Status|MODIFIED|ON_QA

--- Comment #18 from Fedora Update System upda...@fedoraproject.org 
2010-12-04 19:40:20 EST ---
jing-trang-2009-7.fc14 has been pushed to the Fedora 14 testing repository.
 If problems still persist, please make note of it in this bug report.
 If you want to test the update, you can install it with 
 su -c 'yum --enablerepo=updates-testing update jing-trang'.  You can provide
feedback for this update here:
https://admin.fedoraproject.org/updates/jing-trang-2009-7.fc14

-- 
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@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 655601] Review Request: jing-trang - Schema validation and conversion based on RELAX NG

2010-12-03 Thread bugzilla
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=655601

Stanislav Ochotnicky socho...@redhat.com changed:

   What|Removed |Added

   Flag|fedora-review?  |fedora-review+

--- Comment #14 from Stanislav Ochotnicky socho...@redhat.com 2010-12-03 
07:15:54 EST ---
Package seems good now, builds fine and is all in all very nice :-)

APPROVED

-- 
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@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 655601] Review Request: jing-trang - Schema validation and conversion based on RELAX NG

2010-12-03 Thread bugzilla
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=655601

Ville Skyttä ville.sky...@iki.fi changed:

   What|Removed |Added

   Flag||fedora-cvs?

--- Comment #15 from Ville Skyttä ville.sky...@iki.fi 2010-12-03 12:06:30 EST 
---
New Package SCM Request
===
Package Name: jing-trang
Short Description: Schema validation and conversion based on RELAX NG
Owners: scop
Branches: f14
InitialCC:

-- 
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@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 655601] Review Request: jing-trang - Schema validation and conversion based on RELAX NG

2010-12-03 Thread bugzilla
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=655601

--- Comment #16 from Jason Tibbitts ti...@math.uh.edu 2010-12-03 15:40:08 EST 
---
Git done (by process-git-requests).

-- 
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@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 655601] Review Request: jing-trang - Schema validation and conversion based on RELAX NG

2010-12-03 Thread bugzilla
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=655601

--- Comment #17 from Fedora Update System upda...@fedoraproject.org 
2010-12-03 17:20:17 EST ---
jing-trang-2009-7.fc14 has been submitted as an update for Fedora 14.
https://admin.fedoraproject.org/updates/jing-trang-2009-7.fc14

-- 
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@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 655601] Review Request: jing-trang - Schema validation and conversion based on RELAX NG

2010-12-03 Thread bugzilla
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=655601

Fedora Update System upda...@fedoraproject.org changed:

   What|Removed |Added

 Status|ASSIGNED|MODIFIED

-- 
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@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 655601] Review Request: jing-trang - Schema validation and conversion based on RELAX NG

2010-12-02 Thread bugzilla
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=655601

--- Comment #13 from Ville Skyttä ville.sky...@iki.fi 2010-12-02 11:40:00 EST 
---
http://scop.fedorapeople.org/packages/jing-trang.spec
http://scop.fedorapeople.org/packages/jing-trang-2009-7.fc14.src.rpm
http://koji.fedoraproject.org/koji/taskinfo?taskID=2639853

- Put Xalan instead of Saxon in build path (regression in -6).
- Build with OpenJDK.

Comment 12 was correct, but it's not enough to fix the build with GCJ, there
are other issues.  This one's built with OpenJDK, I don't plan to spend time to
get the build working with GCJ.

-- 
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@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 655601] Review Request: jing-trang - Schema validation and conversion based on RELAX NG

2010-12-01 Thread bugzilla
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=655601

--- Comment #11 from Stanislav Ochotnicky socho...@redhat.com 2010-12-01 
08:10:00 EST ---
Hmm, the build fails in my local mock and even koji now:

http://koji.fedoraproject.org/koji/taskinfo?taskID=2636658

-- 
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@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 655601] Review Request: jing-trang - Schema validation and conversion based on RELAX NG

2010-12-01 Thread bugzilla
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=655601

--- Comment #12 from Ville Skyttä ville.sky...@iki.fi 2010-12-01 18:18:31 EST 
---
Hm, it builds fine for F-13 and F-14 mock here, but IIRC it ends up being built
with openjdk for me and not gcj like the koji log indicates.

There's a leftover buglet from my Saxon tries which I suppose may cause this;
export CLASSPATH=$(build-classpath saxon) should be replaced by export
CLASSPATH=$(build-classpath xalan-j2) at start of %build.  Can't test right
now, but probably can tomorrow.

-- 
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@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 655601] Review Request: jing-trang - Schema validation and conversion based on RELAX NG

2010-11-30 Thread bugzilla
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=655601

--- Comment #8 from Stanislav Ochotnicky socho...@redhat.com 2010-11-30 
04:27:14 EST ---
(In reply to comment #7)
  1. (optional) jing-trang-prepare-tarball.sh from srpm can be removed from 
  srpm
 
 All the packages I've worked with include it in the source rpm.  It doesn't
 cost anything to have it there, and the upside is that people can actually
 verify the contents of the srpm as well as roll updates just with the
 information in it without having to hunt missing bits from elsewhere.

Sure, no problem.

  2. simplify documentation installation by using %doc macro
 
 I intentionally didn't do that initially because for some reason I remembered
 that the special %doc macro would do a cp --parents and would thus include
 unwanted subdirs below the doc dirs.  But that's incorrect, simplified in -5.

Good

  3. remove %clean section and rm -rf from %install section
 
 These (and the Group tags) are intentionally left there because I want the
 package to be cleanly buildable as is for example on EL-5 (assuming
 dependencies are available). They don't cause any problems, and I can easily
 live with them around.

I though that might be the case. OK, no problem then.

  I must say I am impressed with the patch and bugs you filed upstream to fix
  the build system a bit. Great work.
 
 Thanks, and thanks for the review which got quite a bit bigger than what you
 initially signed up for.  Next revision with the %doc stuff simplified:
 
 http://scop.fedorapeople.org/packages/jing-trang.spec
 http://scop.fedorapeople.org/packages/jing-trang-2009-5.fc14.src.rpm

Looking into this now.

 Some things I plan to look into whether they'd make sense and be feasible to
 implement in the future for this package, let me know if you'd like them or
 some of them done during the review, or have any opinions about them, in no
 particular order:
 
 - Try to port the missing test suite generation bits to some other XSLT
   processor than the old saxon, and if successful, run the whole test suite.

I think the whole building process for Fedora and other distributions would be
better off with no XSLT generation at all...but maybe I am wrong, since I just
had a glimpse at what you had to do to make this work. 

 - Install dtdinst's schemas and XSLs as non-%doc and register them
   with the system XML catalog.

Yes this should happen too. You probably know how to do it, but just in
case...you can look at log4j spec file which installs log4j dtds

 - Review jing's %docs; isorelax.copying.txt and xerces.copying.txt
   could/should maybe be dropped because they're not shipped in this package
   (and copying.html maybe modified not to link to them) and maybe
   datatype-sample.jar should be dropped as well.

Good idea, also asking upstream about it might help.

-- 
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@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 655601] Review Request: jing-trang - Schema validation and conversion based on RELAX NG

2010-11-30 Thread bugzilla
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=655601

--- Comment #9 from Stanislav Ochotnicky socho...@redhat.com 2010-11-30 
05:20:45 EST ---
Ok, sorry about not noticing it right away, but javadoc sub-package doesn't
include license. There was recent change in packaging guidelines (~2-3 months
ago) that requires all independent packages to include license. Since your
javadoc doesn't Require either of the other sub-packages it has to include
license.

Fix that and you have my +.

-- 
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@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 655601] Review Request: jing-trang - Schema validation and conversion based on RELAX NG

2010-11-29 Thread bugzilla
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=655601

--- Comment #6 from Stanislav Ochotnicky socho...@redhat.com 2010-11-29 
08:33:07 EST ---
Package Review
==

Key:
- = N/A
x = Check
! = Problem
? = Not evaluated

=== REQUIRED ITEMS ===
[x]  Rpmlint output:
dtdinst.noarch: W: no-manual-page-for-binary dtdinst
jing.noarch: W: spelling-error Summary(en_US) validator - invalidator,
validation, validate
jing.noarch: W: spelling-error %description -l en_US validator - invalidator,
validation, validate
jing.noarch: W: dangling-symlink /usr/share/doc/jing-2009-4.fc15/doc/api
/usr/share/javadoc/jing
jing.noarch: W: no-manual-page-for-binary jing
jing-trang.src: W: strange-permission jing-trang-prepare-tarball.sh 0755L
jing-trang.src: W: invalid-url Source0: jing-trang-2009.tar.xz
trang.noarch: W: spelling-error Summary(en_US) Multi - Mulch, Mufti
trang.noarch: W: no-manual-page-for-binary trang
5 packages and 0 specfiles checked; 0 errors, 9 warnings.

The jing-trang-prepare-tarball.sh doesn't have to be included in the srpm since
it already includes the tarball. Generally we just include it in repository
close to spec file

[x]  Package is named according to the Package Naming Guidelines[1].
[x]  Spec file name must match the base package name, in the format
%{name}.spec.
[x]  Package meets the Packaging Guidelines[2].
[x]  Package successfully compiles and builds into binary rpms.
[!]  Buildroot definition is not present
According to new guidelines Buildroot definitions are to be omitted
(automatically provided by recent rpmbuild)
[x]  Package is licensed with an open-source compatible license and meets other
legal requirements as defined in the legal section of Packaging
Guidelines[3,4].
[x]  License field in the package spec file matches the actual license.
License type: BSD
[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.

But why do all the (from my POV) unnecessary directory creating in %{_docdir}
instead of using %doc macro in %files section. No need to do all that copying
during %install
Something like this can do the trick:
--
%doc readme.html doc/ sample/
--
[x]  All independent sub-packages have license of their own
[x]  Spec file is legible and written in American English.
[x]  Sources used to build the package matches the upstream source, as provided
in the spec URL.
[x]  All build dependencies are listed in BuildRequires, except for any that
are listed in the exceptions section of Packaging Guidelines[5].
[x]  Package must own all directories that it creates.
[x]  Package requires other packages for directories it uses.
[x]  Package does not contain duplicates in %files.
[x]  Permissions on files are set properly.
[!]  Package does NOT have a %clean section which contains rm -rf %{buildroot}
(or $RPM_BUILD_ROOT). (not needed anymore)
%clean and rm -rf at the beginning of %install is not needed and can be omitted
[x]  Package consistently uses macros (no %{buildroot} and $RPM_BUILD_ROOT
mixing)
[x]  Package contains code, or permissable content.
[x]  Fully versioned dependency in subpackages, if present.
[-]  Package contains a properly installed %{name}.desktop file if it is a GUI
application.
[x]  Package does not own files or directories owned by other packages.
[x]  Javadoc documentation files are generated and included in -javadoc
subpackage
[x]  Javadocs are placed in %{_javadocdir}/%{name} (no -%{version} symlinks)
[x]  Packages have proper BuildRequires/Requires on jpackage-utils
[x]  Javadoc subpackages have Require: jpackage-utils
Not really but through a dependency..
[-]  Package uses %global not %define
[x]  If package uses tarball from VCS include comment how to re-create that
tarball (svn export URL, git clone URL, ...)
[x]  If source tarball includes bundled jar/class files these need to be
removed prior to building
[x]  All filenames in rpm packages must be valid UTF-8.
[x]  Jar files are installed to %{_javadir}/%{name}.jar (see [6] for details)
[-]  If package contains pom.xml files install it (including depmaps) even when
building with ant
[-]  pom files has correct add_to_maven_depmap call which resolves to the pom
file (use JPP. and JPP- correctly)


=== Other suggestions ===
[x]  If possible use upstream build method (maven/ant/javac)
[x]  Avoid having BuildRequires on exact NVR unless necessary
[x]  Package has BuildArch: noarch (if possible)
[x]  Latest version is packaged.
[x]  Reviewer should test that the package builds in mock.
Tested on: fedora-rawhide-x86_64


=== Issues ===
1. (optional) jing-trang-prepare-tarball.sh from srpm can be removed from srpm
2. simplify documentation installation by using %doc macro
3. remove %clean section and rm -rf from %install section
Other than these three things, 

[Bug 655601] Review Request: jing-trang - Schema validation and conversion based on RELAX NG

2010-11-29 Thread bugzilla
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=655601

--- Comment #7 from Ville Skyttä ville.sky...@iki.fi 2010-11-29 13:41:13 EST 
---
 1. (optional) jing-trang-prepare-tarball.sh from srpm can be removed from srpm

All the packages I've worked with include it in the source rpm.  It doesn't
cost anything to have it there, and the upside is that people can actually
verify the contents of the srpm as well as roll updates just with the
information in it without having to hunt missing bits from elsewhere.

 2. simplify documentation installation by using %doc macro

I intentionally didn't do that initially because for some reason I remembered
that the special %doc macro would do a cp --parents and would thus include
unwanted subdirs below the doc dirs.  But that's incorrect, simplified in -5.

 3. remove %clean section and rm -rf from %install section

These (and the Group tags) are intentionally left there because I want the
package to be cleanly buildable as is for example on EL-5 (assuming
dependencies are available). They don't cause any problems, and I can easily
live with them around.

 I must say I am impressed with the patch and bugs you filed upstream to fix
 the build system a bit. Great work.

Thanks, and thanks for the review which got quite a bit bigger than what you
initially signed up for.  Next revision with the %doc stuff simplified:

http://scop.fedorapeople.org/packages/jing-trang.spec
http://scop.fedorapeople.org/packages/jing-trang-2009-5.fc14.src.rpm

Some things I plan to look into whether they'd make sense and be feasible to
implement in the future for this package, let me know if you'd like them or
some of them done during the review, or have any opinions about them, in no
particular order:

- Try to port the missing test suite generation bits to some other XSLT
  processor than the old saxon, and if successful, run the whole test suite.
- Install dtdinst's schemas and XSLs as non-%doc and register them
  with the system XML catalog.
- Review jing's %docs; isorelax.copying.txt and xerces.copying.txt
  could/should maybe be dropped because they're not shipped in this package
  (and copying.html maybe modified not to link to them) and maybe
  datatype-sample.jar should be dropped as well.

-- 
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@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 655601] Review Request: jing-trang - Schema validation and conversion based on RELAX NG

2010-11-28 Thread bugzilla
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=655601

Ville Skyttä ville.sky...@iki.fi changed:

   What|Removed |Added

Summary|Review Request: jing -  |Review Request: jing-trang
   |RELAX NG validator in Java  |- Schema validation and
   ||conversion based on RELAX
   ||NG

--- Comment #4 from Ville Skyttä ville.sky...@iki.fi 2010-11-28 10:02:03 EST 
---
http://scop.fedorapeople.org/packages/jing-trang.spec
http://scop.fedorapeople.org/packages/jing-trang-2009-4.fc13.src.rpm

It took some effort to get this built from upstream svn checkout, but here
goes.  Unfortunately not all of the test suite could be done because
(generating) it has dependencies to an old version of saxon which we don't
have.

All rpmlint messages are ignorable:

dtdinst.noarch: W: no-manual-page-for-binary dtdinst
jing-trang.src: W: strange-permission jing-trang-prepare-tarball.sh 0755L
jing-trang.src: W: invalid-url Source0: jing-trang-2009.tar.xz
jing.noarch: W: spelling-error Summary(en_US) validator - invalidator,
validation, validate
jing.noarch: W: spelling-error %description -l en_US validator - invalidator,
validation, validate
jing.noarch: W: dangling-symlink /usr/share/doc/jing-2009-4.fc14/doc/api
/usr/share/javadoc/jing
jing.noarch: W: no-manual-page-for-binary jing
trang.noarch: W: spelling-error Summary(en_US) Multi - Mulch, Mufti
trang.noarch: W: no-manual-page-for-binary trang
5 packages and 0 specfiles checked; 0 errors, 9 warnings.

-- 
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@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 655601] Review Request: jing-trang - Schema validation and conversion based on RELAX NG

2010-11-28 Thread bugzilla
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=655601

--- Comment #5 from Ville Skyttä ville.sky...@iki.fi 2010-11-28 10:03:34 EST 
---
*** Bug 655581 has been marked as a duplicate of this bug. ***

-- 
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@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review