[Bug 1413434] Review Request: php-gettext-languages - Generate gettext language lists with plural rules

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

 



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



--- Comment #2 from Randy Barlow <randy@xxxxxxxxxxxxxxxxxxxxx> ---
Hello Remi!

The upstream project already has an autoloader.php for the installed package,
and they had a tests/bootstrap.php that was close to what we needed but not
quite the same. I ended up using two sed statements to modify the
tests/bootstrap.php and bin/export.php so that they require_once with the
installed path. What do you think about that approach?

Also, I realized that it seems that they may expect that bin/export.{php,sh} to
get distributed with the package and I have not done that so far because the
name "export.php" sounds too general to go into /usr/bin. Do you think we
should distribute that executable? If so, should I name it something more
specific (gettext-languages-export.sh, cldr-to-gettext-plural-rules-export.sh?)
The tests rely on this executable running first to generate some data file (see
tests/bootstrap.php), which is why I used that sed statement, but if we want to
distribute it I'll need to sed a copy of it instead (and sed the one we install
separately).

Here's what I have so far, without distributing that executable:

Spec URL: https://bowlofeggs.fedorapeople.org/php-gettext-languages.spec
SRPM URL:
https://bowlofeggs.fedorapeople.org/php-gettext-languages-2.1.3-2.fc26.src.rpm

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
_______________________________________________
package-review mailing list -- package-review@xxxxxxxxxxxxxxxxxxxxxxx
To unsubscribe send an email to package-review-leave@xxxxxxxxxxxxxxxxxxxxxxx




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