[Bug 646663] Rename review: drupal -> drupal6, F15+

[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=646663

Sven Lankes <sven@xxxxxxx> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
               Flag|                            |needinfo?(limb@xxxxxxxxxxxx
                   |                            |)

--- Comment #3 from Sven Lankes <sven@xxxxxxx> 2010-12-20 17:32:38 EST ---
Most importantly: The spec is missing a pair of provides/obsoletes here.

rpmlint output:

> drupal6.noarch: W: spelling-error %description -l en_US weblogs -> we blogs, > we-blogs, web logs
> drupal6.noarch: W: spelling-error %description -l en_US Drupal -> Drupelet, Drupe, Druidical
> drupal6.noarch: W: spelling-error %description -l en_US skinnable -> winnable, scannable, skinniness

Those can be ignored

> drupal6.noarch: W: dangling-relative-symlink /usr/share/drupal6/sites ../../../etc/drupal6

Needed here.

> drupal6.noarch: E: non-standard-dir-perm /var/lib/drupal6/files/default 0775L
> drupal6.noarch: E: non-standard-dir-perm /var/lib/drupal6 0775L
> drupal6.noarch: E: non-standard-dir-perm /var/lib/drupal6/files 0775L

Those need to have group access for the web-server.

> drupal6.noarch: E: executable-marked-as-config-file /etc/cron.hourly/drupal6

The content of this is not going to change so marking it as config-file seems
fine to me.

> drupal6.noarch: E: htaccess-file /usr/share/drupal6/.htaccess

Hmm. Is that .htaccess-file really neccessary there? I just removed the link on
my test vm and things (frontend and backend) seem to be working as before.

-- 
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]