Bug 158598

Summary: apparmor-utils lacking dependency on vim breaks vim install
Product: [openSUSE] SUSE Linux 10.1 Reporter: Torsten Duwe <duwe>
Component: AppArmorAssignee: Jesse Michael <jmichael>
Status: RESOLVED FIXED QA Contact: Dominic W Reynolds <dreynolds>
Severity: Blocker    
Priority: P5 - None CC: suse-beta
Version: Beta 8   
Target Milestone: ---   
Hardware: Other   
OS: Linux   
Whiteboard:
Found By: Development Services Priority:
Business Priority: Blocker: ---
Marketing QA Status: --- IT Deployment: ---

Description Torsten Duwe 2006-03-16 11:38:08 UTC 
apparmor-utils tries to install /usr/share/vim/current/syntax/apparmor.vim
as part of its file list. The directory /usr/share/vim/current however is a symlink to the current version of vim installed, once it is installed.

If apparmor-utils gets installed before vim, vim installation fails because the symlink cannot be unpacked. Ignoring RPM errors like this one generally is a bad idea[tm].

Fix: if apparmor-utils wants to add something to a vim, and expects it to exist, it "Requires: vim".
Comment 1 Torsten Duwe 2006-03-16 11:58:34 UTC 
It might even be a PreReq.
Comment 2 Dominic W Reynolds 2006-03-21 07:00:39 UTC 
This seems like something that could lead to other dependency problems (what are the recursive set of dependencies for vim?).

Are rpm dependencies ignored during installation of the base OS? If not how is installation occuring? On beta 8 i have vim and apparmor-utils.
 
In what sense is this a blocker?

I spoke with the maintainer (jmichael) who suggested that we remove this syntax file and add it to the vim package. 

We will try to get that accomplished before the next drop. Ok to lower this from a blocker to major?
Comment 3 Torsten Duwe 2006-03-21 12:56:06 UTC 
I consider this a blocker because it can make rpm install return with an error. Depending on your installer you may end up with an incomplete system. As I wrote, ignoring errors like this is a bad idea, because far more serious errors will manifest themselves in the same way.

It seems that because of the current CD order this problem doesn't show, but that's purely a game of chance.
Comment 4 Andreas Jaeger 2006-03-23 19:29:38 UTC 
Dom, please add the file to vim.
Comment 5 Jesse Michael 2006-03-27 15:56:46 UTC 
we've removed the apparmor syntax file from the apparmor-utils package and have submitted an updated package to autobuild to get rid of the blocker.

we'll submit a patch to add the syntax file to the suse vim package maintainer.