[Bug 784201] Review Request: vim-nerdtree - A tree explorer plugin that owns your momma

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

--- Comment #2 from Vít Ondruch <vondruch@xxxxxxxxxx> 2012-01-25 08:18:09 EST ---
* Should use vim-common
  - It seems that instead of requiring vim and vim-filesystem, you should use
    vim-common package.

* Do not own the directories owned by vim-filesystem.
  - According to FPG:

    Directory ownership is a little more complex than file ownership. Packages
    must own all directories they put files in, except for:

      any directories owned by the filesystem, man, or other explicitly created
     -filesystem packages

  - You should own just content of the following directories, not the
    directories itself: 

    %{vimfiles_root}/doc
    %{vimfiles_root}/plugin
    %{vimfiles_root}/syntax


* You shoud preserve the file timestamps
  - Please use cp -a or install command

* Package naming
  - I am unsure it the package name is really correct. Could you please check
    with upstream if they are going to stick with "nerdtree" from now?

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