Bug 158598 - apparmor-utils lacking dependency on vim breaks vim install
Summary: apparmor-utils lacking dependency on vim breaks vim install
Status: RESOLVED FIXED
Alias: None
Product: SUSE Linux 10.1
Classification: openSUSE
Component: AppArmor (show other bugs)
Version: Beta 8
Hardware: Other Linux
: P5 - None : Blocker (vote)
Target Milestone: ---
Assignee: Jesse Michael
QA Contact: Dominic W Reynolds
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2006-03-16 11:38 UTC by Torsten Duwe
Modified: 2006-03-27 15:56 UTC (History)
1 user (show)

See Also:
Found By: Development
Services Priority:
Business Priority:
Blocker: ---
Marketing QA Status: ---
IT Deployment: ---


Attachments

Note You need to log in before you can comment on or make changes to this bug.
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.