Spec URL: http://wilqu.fr/rpms/tt-rss/tt-rss.spec SRPM URL: http://wilqu.fr/rpms/tt-rss/tt-rss-1.5.11-1.fc17.src.rpm Description: Tiny Tiny RSS is an open source web-based news feed (RSS/Atom) reader and aggregator, designed to allow you to read news from any location, while feeling as close to a real desktop application as possible. Fedora Account System Username: wilqu $ rpmlint tt-rss-1.5.11-1.fc17.src.rpm tt-rss-1.5.11-1.fc17.noarch.rpm tt-rss.src: I: enchant-dictionary-not-found en_US tt-rss.noarch: W: non-etc-or-var-file-marked-as-conffile /usr/share/tt-rss/config.php 2 packages and 0 specfiles checked; 0 errors, 1 warnings. The configuration file is expected to be at this location, it can not be moved to /etc or /var.
Moved the configuration file to /etc/tt-rss/config.php Spec URL: http://wilqu.fr/rpms/tt-rss/tt-rss.spec SRPM URL: http://wilqu.fr/rpms/tt-rss/tt-rss-1.5.11-2.fc17.src.rpm $ rpmlint tt-rss-1.5.11-2.fc17.src.rpm tt-rss-1.5.11-2.fc17.noarch.rpm tt-rss.src: I: enchant-dictionary-not-found en_US 2 packages and 0 specfiles checked; 0 errors, 0 warnings.
This package have some issues to be fixed and is not yet ready.
With Google announcing the shutdown of Google Reader, I'm sure more people will be interested in running tt-rss on their own server. Have you had a chance to update your packages for review?
I didn't notice that php-htmlpurifier-htmlpurifier was in the repositories. I'll update the package probably this week end. I moved the files to my fedorapeople: Spec URL: http://wilqu.fedorapeople.org/reviews/tt-rss/tt-rss.spec SRPM URL: http://wilqu.fedorapeople.org/reviews/tt-rss/tt-rss-1.5.11-2.fc17.src.rpm
Spec URL: http://wilqu.fedorapeople.org/reviews/Tiny-Tiny-RSS/Tiny-Tiny-RSS.spec SRPM URL: http://wilqu.fedorapeople.org/reviews/Tiny-Tiny-RSS/Tiny-Tiny-RSS-1.7.4-1.fc18.src.rpm I updated the package to the last version and removed some bundled libs, but there are some bundled libs left that are not packaged yet. One of them (jsmin) doesn't have an acceptable license, so it has to be removed. For now, the package requires both mysql and postgresql bindings, but I'll probably split it to avoid that. The name of the tarball changed, so I also changed the name of the package.
Hello, thanks for your work. I'm trying the package and I have problems to run the update_daemon2.php, I get this error: PHP Parse error: syntax error, unexpected '';' (T_CONSTANT_ENCAPSED_STRING) in /usr/share/Tiny-Tiny-RSS/include/functions.php on line 3653 That line is: $sphinxClient->SetServer('localhost', 9312); so it seems that something is broken when removing the bundled sphinx library. With the vanilla tar.gz it worked for me. I use this unit file to launch the update daemon. It may be interesting to include something similar in your package: [Unit] Description=Tiny Tiny RSS update daemon After=network.target mysqld.service Requires=mysqld.service [Service] User=apache Group=apache WorkingDirectory=/usr/share/Tiny-Tiny-RSS Type=simple ExecStart=/usr/bin/php ./update_daemon2.php PrivateTmp=true InaccessibleDirectories=/home /root /boot /opt /mnt /media ReadOnlyDirectories=/etc /usr [Install] WantedBy=multi-user.target
/etc/Tiny-Tiny-RSS/config.php contains the database password in clear text, so it should have apache:apache 0600 permissions. Also, you are including some files twice. When you indicate a directory, all files are added recursively (unless you use %dir): /etc/Tiny-Tiny-RSS/config.php /usr/share/Tiny-Tiny-RSS/cache/export /usr/share/Tiny-Tiny-RSS/cache/images /usr/share/Tiny-Tiny-RSS/feed-icons /usr/share/Tiny-Tiny-RSS/lock In the dependencies, you should require policycoreutils-python instead of restorecon and semanage.
Created attachment 712646 [details] Fix the sphinx unbundled library issue You forgot a ' in the sed
Spec URL: http://wilqu.fedorapeople.org/reviews/Tiny-Tiny-RSS/Tiny-Tiny-RSS.spec SRPM URL: http://wilqu.fedorapeople.org/reviews/Tiny-Tiny-RSS/Tiny-Tiny-RSS-1.7.4-2.fc18.src.rpm Thanks for your comments. I fixed your issues plus many more. Actually the package wasn't functionnal at all. This time I tested it (with postgresql) and it should work. I include files twice to add special tags (%config(noreplace), %attr). Is there a better way to do it? There is still work to do. Here is the list of libs to be packaged: * accept-to-gettext (couldn't find upstream site) * MiniTemplator http://www.source-code.biz/MiniTemplator/ https://bugzilla.redhat.com/show_bug.cgi?id=926968 * Mobile_Detect https://github.com/serbanghita/Mobile-Detect * otphp https://github.com/lelag/otphp * phpqrcode http://phpqrcode.sourceforge.net/ * pubsubhubbub https://code.google.com/p/pubsubhubbub/
I don't see any files included twice.
I don't think I'll have the patience to package and maintain this. I'm closing this review request.