Bug 242313 - Review Request: phpTodo - PHP todo list manager
Review Request: phpTodo - PHP todo list manager
Product: Fedora
Classification: Fedora
Component: Package Review (Show other bugs)
All Linux
low Severity medium
: ---
: ---
Assigned To: Hans de Goede
Fedora Package Reviews List
Depends On:
  Show dependency treegraph
Reported: 2007-06-03 03:50 EDT by Marc Bradshaw
Modified: 2007-11-30 17:12 EST (History)
1 user (show)

See Also:
Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of:
Last Closed: 2007-09-24 19:21:05 EDT
Type: ---
Regression: ---
Mount Type: ---
Documentation: ---
Verified Versions:
Category: ---
oVirt Team: ---
RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: ---
hdegoede: fedora‑review+
kevin: fedora‑cvs+

Attachments (Terms of Use)

  None (edit)
Description Marc Bradshaw 2007-06-03 03:50:06 EDT
First package, sponsor required.

Spec URL: http://marcbradshaw.co.uk/packages/SPECS/phpTodo.spec

SRPM URL: http://marcbradshaw.co.uk/packages/SRPMS/phpTodo-0.8.1-0.3.beta.fc6.src.rpm

The general idea behind this program is to keep all of your todo lists in
one central location.  I've grown very tired of having lists all over the
place and being unable to reconcile them...
Comment 1 Marc Bradshaw 2007-06-05 00:51:17 EDT
Added some SELinux stuff, new SRPM URL:
Comment 2 Tyler Owen 2007-06-16 12:25:45 EDT
In a quick look at your SPEC I noticed that the Source0 URL does not meet the
standard for Sourceforge projects. 

should be: 

Source0: http://downloads.sourceforge.net/%{name}/%{name}-%{version}.tar.gz

Comment 3 Marc Bradshaw 2007-06-17 19:13:41 EDT
Thanks Tyler, url has been changed to
http://downloads.sourceforge.net/phptodo/%{name}-%{version}beta.tar.gz which is
as close to the standard as is possible for that url %{name} being phpTodo
rather than the lowercase version.

New SRPM: http://marcbradshaw.co.uk/packages/SRPMS/phpTodo-0.8.1-0.5.beta.f7.src.rpm

I think the SElinux parts still need some work, if somebody with the knowhow
could take a look at that section I would appreciate it.
Comment 7 Hans de Goede 2007-09-19 14:16:31 EDT
I've taken a look at the last version and it looks good, including the selinux
rules, but then I installed it, started httpd (I've it disabled by default) and
then went to localhost/phpTodo, and all I get is a white page, view source shows
that nothing was send. Do I need to do anything else?
Comment 8 Marc Bradshaw 2007-09-19 19:38:47 EDT
There is a mysql database which needs to be setup but having that missing should
not result in a blank page being sent, a useful error should be displayed instead.
What selinux level are you running at?  Is there anything useful in
Comment 9 Hans de Goede 2007-09-20 02:20:14 EDT
(In reply to comment #8)
> There is a mysql database which needs to be setup but having that missing should
> not result in a blank page being sent, a useful error should be displayed instead.
> What selinux level are you running at?

I tried using setenforce 0, didn't help

>  Is there anything useful in
> /var/log/httpd/error_log?
I'll check tonight when I'm back behind my home workstation.

Comment 10 Marc Bradshaw 2007-09-20 02:55:43 EDT
I installed it in a fresh VM running F7 and I think I have found the problem.
The location of the php-Smarty install changed between FC6 and F7 and the RPM
uses the old version.  I tested it on 2 installs, one was pre F7 and the other
used pre F7 php RPMs as the stuff being worked on there isn't php5 compatible.
I will have a fix done shortly.
Comment 11 Marc Bradshaw 2007-09-20 06:53:40 EDT
I have set the sample config file to default to the F7 location of Smarty and
check for the FC6 location as a backup.
The error message received when the database has not been setup has also been
changed to be more useful.

Comment 12 Hans de Goede 2007-09-20 13:44:24 EDT
Ah yes, better now, now it works with selinux disabled, to get it to "work"
(only tested up to the please configure page) with selinux enabled. make the
following changes:
-the correct selinux type for php and normal files is:
 httpd_sys_content_t, the type you now use is for old fashioned cgi scripts,
 not for escripted language handled through apache modules.
-add /etc/phpTodo/config.php to the list of files that need a selinux type set
 (extra semanage command and add to the restorecon cmdline)
Comment 13 Marc Bradshaw 2007-09-22 06:14:22 EDT
Latest SRPM: http://marcbradshaw.co.uk/packages/phpTodo-0.8.1-0.8.beta.src.rpm

selinux changed as above and /var/cache/phpTodo/ also added to selinux.
the initial database schema has also been patched such that it works with the
fedora supplied version of mySQL.
Comment 14 Hans de Goede 2007-09-22 09:25:38 EDT
Looks good, approved!

I'm ready to sponsor you now, please create a Fedora Account and request CVS
extras group membership, then I'll ack that.
Comment 15 Marc Bradshaw 2007-09-22 11:15:51 EDT
Thanks, cvsextras group has been requested.
Comment 16 Hans de Goede 2007-09-22 13:03:01 EDT
Have you filled in the CLA? Also, what is your fedora account system username?
Comment 17 Marc Bradshaw 2007-09-22 20:15:32 EDT
Yes, cla has been done, username is deebs
Comment 18 Hans de Goede 2007-09-23 01:40:06 EDT
Ah, yes with a username like that knowing your username helps. You have been
sponsored. You can file CVS admin requests for creation of the CVS modules for
your packages now.
Comment 19 Marc Bradshaw 2007-09-23 07:37:46 EDT
New Package CVS Request
Package Name: phpTodo
Short Description: PHP todo list manager
Owners: deebs
Branches: F-7
Cvsextras Commits: yes
Comment 20 Kevin Fenzi 2007-09-24 12:14:39 EDT
cvs done.

Note You need to log in before you can comment on or make changes to this bug.