Bug 823065

Updates per comments in bug 823043

- Removed BuildRoot
- Changed php require to php-common
- Added the following requires based on phpci results:
  php-libxml, php-pcre, php-simplexml
- Removed %defattr from %files section

SPEC URL:
http://people.redhat.com/siwinski/rpmbuild/SPECS/php-symfony2-Translation.spec

SRPM URL:
http://people.redhat.com/siwinski/rpmbuild/SRPMS/php-symfony2-Translation-2.0.14-2.fc16.src.rpm

Update per https://bugzilla.redhat.com/show_bug.cgi?id=823041#c5

- Moved documentation to correct location

SPEC URL:
http://people.redhat.com/siwinski/rpmbuild/SPECS/php-symfony2-Translation.spec

SRPM URL:
http://people.redhat.com/siwinski/rpmbuild/SRPMS/php-symfony2-Translation-2.0.14-3.fc16.src.rpm

Updated to upstream version 2.0.15 & updates per bug #817303

- Removed "BuildRequires: php-pear >= 1:1.4.9-1.2"
- Updated %prep section
- Removed cleaning buildroot from %install section
- Removed documentation move from %install section (fixed upstream)
- Removed %clean section
- Updated %doc in %files section

SPEC URL: http://people.redhat.com/siwinski/rpmbuild/SPECS/php-symfony2-Translation.spec

SRPM URL: http://people.redhat.com/siwinski/rpmbuild/SRPMS/php-symfony2-Translation-2.0.15-1.fc16.src.rpm

Issue: XSD files installed as doc, but referenced by code.

-- XSD files --
* Symfony/Component/Translation/Loader/schema/dic/xliff-core/xliff-core-1.2-strict.xsd
* Symfony/Component/Translation/Loader/schema/dic/xliff-core/xml.xsd

-- Code (from Translation/Loader/XliffFileLoader.php) --
    private function parseFile($file)
    {
     	$dom = new \DOMDocument();
        $current = libxml_use_internal_errors(true);
        if (!@$dom->load($file, defined('LIBXML_COMPACT') ? LIBXML_COMPACT : 0)) {
            throw new \RuntimeException(implode("\n", $this->getXmlErrors()));
        }

	$location = str_replace('\\', '/', __DIR__).'/schema/dic/xliff-core/xml.xsd';
       	$parts = explode('/', $location);
        if (0 === stripos($location, 'phar://')) {
            $tmpfile = tempnam(sys_get_temp_dir(), 'sf2');
            if ($tmpfile) {
               	copy($location, $tmpfile);
               	$parts = explode('/', str_replace('\\', '/', $tmpfile));
            }
        }
     	$drive = '\\' === DIRECTORY_SEPARATOR ? array_shift($parts).'/' : '';
        $location = 'file:///'.$drive.implode('/', array_map('rawurlencode', $parts));

        $source = file_get_contents(__DIR__.'/schema/dic/xliff-core/xliff-core-1.2-strict.xsd');
	$source = str_replace('http://www.w3.org/2001/xml.xsd', $location, $source);

        if (!@$dom->schemaValidateSource($source)) {
            throw new \RuntimeException(implode("\n", $this->getXmlErrors()));
        }
        $dom->validateOnParse = true;
        $dom->normalizeDocument();
	libxml_use_internal_errors($current);

        return simplexml_import_dom($dom);
    }



See original finding of this issue at https://bugzilla.redhat.com/show_bug.cgi?id=823046#c8

Added optional requires

- Added php-pear(%{pear_channel}/Config) require
- Removed ownership for directories already owned by required packages

Update per comment #4

- Fix package.xml for *.xsd files issue

SPEC URL: http://people.redhat.com/~siwinski/rpmbuild/SPECS/php-symfony2-Translation.spec

SRPM URL: http://people.redhat.com/~siwinski/rpmbuild/SRPMS/php-symfony2-Translation-2.0.15-3.fc17.src.rpm

Created attachment 593977 [details]
php-symfony2-Translation-review.txt

Generated by fedora-review 0.1.3

Please consider adding (optionnal dependency)
Requires:         php-pear(%{pear_channel}/Yaml)

No blocker.

== APPROVED ==

(In reply to comment #7)
> Please consider adding (optionnal dependency)
> Requires:         php-pear(%{pear_channel}/Yaml)

Thanks Remi!

I totally missed that optional dependency in Symfony/Component/Translation/Loader/YamlFileLoader.php.  Note to self: "phpci print --recursive --report=class" would have helped out there ;)

I will notify upstream that they may wish to update their package.xml with that optional dependency since it only lists pear.symfony.com/Config.

I will add the dependency after the initial import of the approved package.

New Package SCM Request
=======================
Package Name: php-symfony2-Translation
Short Description: Symfony2 Translation Component
Owners: siwinski
Branches: f16 f17 el6
InitialCC:

Git done (by process-git-requests).

php-symfony2-Translation-2.0.15-4.fc17 has been submitted as an update for Fedora 17.
https://admin.fedoraproject.org/updates/php-symfony2-Translation-2.0.15-4.fc17

php-symfony2-Translation-2.0.15-4.fc16 has been submitted as an update for Fedora 16.
https://admin.fedoraproject.org/updates/php-symfony2-Translation-2.0.15-4.fc16

php-symfony2-Translation-2.0.15-4.el6 has been submitted as an update for Fedora EPEL 6.
https://admin.fedoraproject.org/updates/php-symfony2-Translation-2.0.15-4.el6

php-symfony2-Translation-2.0.15-4.el6 has been pushed to the Fedora EPEL 6 testing repository.

php-symfony2-Translation-2.0.15-4.fc16 has been pushed to the Fedora 16 stable repository.

php-symfony2-Translation-2.0.15-4.fc17 has been pushed to the Fedora 17 stable repository.

php-symfony2-Translation-2.0.15-4.el6 has been pushed to the Fedora EPEL 6 stable repository.