Skip Menu |
 

This queue is for tickets about the YAML-Syck CPAN distribution.

Report information
The Basics
Id: 39572
Status: resolved
Priority: 0/
Queue: YAML-Syck

People
Owner: Nobody in particular
Requestors: SREZIC [...] cpan.org
Cc:
AdminCc:

Bug Information
Severity: Wishlist
Broken in: 1.05
Fixed in: (no value)



Subject: Remove compatibility warning
Download (untitled) / with headers
text/plain 340b
I think the compatibility warning in Makefile.PL can be removed completely if a newer or no YAML::Syck was installed on the system. At least, if you don't want to remove it, then you can choose friendlier words, e.g. "NOTE" instead of "WARNING", and maybe tell the user that s/he already worked with a newer YAML::Syck. Regards, Slaven
Download (untitled) / with headers
text/plain 1.4k
On Wed Sep 24 04:36:55 2008, SREZIC wrote: Show quoted text
> I think the compatibility warning in Makefile.PL can be removed > completely if a newer or no YAML::Syck was installed on the system. At > least, if you don't want to remove it, then you can choose friendlier > words, e.g. "NOTE" instead of "WARNING", and maybe tell the user that > s/he already worked with a newer YAML::Syck.
Yes, probably. Patches would be welcome. (This is a form-reply that isn't specific to your particular report) YAML::Syck has just acquired one new maintainer (me), it still doesn't have anyone that *cares* about it. But I'm willing to help solve your report & release a new version with the fix if it's easy for me. It now has a Git repository at: http://github.com/avar/YAML-Syck If your report is a patch that fixes a problem, great. Please remake the patch against Git by forking that repo and sending me a pull request on GitHub (or an update to this bug if you prefer git-format-patch(1) or some other repo provider..). Make sure to include a test for what you fixed. If your report is some code that fails (and you have a testcase for it) a patch against the test suite to demonstrate that failure would be very useful. It's OK if the test crashes and burns, see Test::More's docs for how to make TODO tests that fail now, but shouldn't. Even if it segfaults perl C<system $^X => qw/ -Mblib -MYAML::Syck .../> or something like that and checking the return value will do.
Fixed in Git.


This service is sponsored and maintained by Best Practical Solutions and runs on Perl.org infrastructure.

Please report any issues with rt.cpan.org to rt-cpan-admin@bestpractical.com.