[Bug 506425] Review Request: lightsquid - a light, small and fast log analyzer for squid proxy

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

 



Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

manuel wolfshant <wolfy@xxxxxxxxxxxxxxxxxx> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|ASSIGNED                    |NEW
                 CC|                            |wolfy@xxxxxxxxxxxxxxxxxx
         AssignedTo|wolfy@xxxxxxxxxxxxxxxxxx    |nobody@xxxxxxxxxxxxxxxxx
               Flag|fedora-review?              |

--- Comment #30 from manuel wolfshant <wolfy@xxxxxxxxxxxxxxxxxx> 2010-07-20 06:09:49 EDT ---
    So, now
- you try to run the cron script as the NONexisting user "lightsquid" ( there
will exist a group named lightsquid, but no user);
- you do not use at anything the group that you create in %pre;
- instead of simply deleting the line 
     %dir %{lightdir}
from the "%files apache" section, you have once again _moved_ ALL the cgi
scripts from lightscript-apache to the main package (which , by the way, does
not need "exclude" lines for the files which are included in lightsquid-apache)
- mind that I did not check what gets installed as default config file, now
that instead of removing the "%{__rm} -rf %{SOURCE1}.redhat" from %post you
decided to nuke all occurrences of %{SOURCE1}.redhat from the spec.

    To be honest, I have doubts that you will be able to properly maintain the
package once imported, therefore I remove myself as package reviewer.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.
_______________________________________________
package-review mailing list
package-review@xxxxxxxxxxxxxxxxxxxxxxx
https://admin.fedoraproject.org/mailman/listinfo/package-review


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