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

[Bug 165686] Review Request: Syck - YAML for Ruby, Python, PHP and OCaml



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

Summary: Review Request: Syck - YAML for Ruby, Python, PHP and OCaml


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





------- Additional Comments From oliver linux-kernel at  2005-08-24 05:05 EST -------
(In reply to comment #1)
> - Only the C library is currently being built.  You have to do more to get the
> scripting language bindings built.  When you do that you probably want to split
> the package up with one for each language binding.

OK. Will do that.

> - The Ruby bindings are included in the FC4 Ruby (1.8.0 was the first version
> that should have it.)

Which means, that I don't have to build it? Looking at the ruby 1.8.2 package, I
cannot find the yaml extension... (But normally I don't have anything to do with
ruby).

> - On x86_64, make check for the main library seems to be having problems.
> YtsSpecificationExamples_27 seems to go on forever (10 minutes wallclock before
> I hit Ctrl-C.)

Bad. But as I don't have a x86_64 around, it's hard to find out what's
happening. :-/

> - The spec file looks nothing like the fedora rpm spectemplate (available from
> fedora-rpmdevtools).  Some things undoubtably don't matter but others have very
> obviously been left out (for instance the ldconfig call in %post/%postun).

You are correct, I'm a bad guy; I should have cleaned the spec before submitting
it...

> Running the package through the packaging guidelines
> http://www.fedoraproject.org/wiki/PackagingGuidelines
> or the guidelines "cheat sheet":
> http://www.fedoraproject.org/wiki/PackageReviewGuidelines
> 
> would be a good idea so the spec doesn't leave reviewers trying to figure out
> where to start :-)

Mea culpa.

-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug, or are watching someone who is.


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