Index: features/core/src/main/java/org/apache/karaf/features/internal/FeaturesServiceImpl.java =================================================================== --- features/core/src/main/java/org/apache/karaf/features/internal/FeaturesServiceImpl.java (revision 1139116) +++ features/core/src/main/java/org/apache/karaf/features/internal/FeaturesServiceImpl.java (working copy) @@ -350,11 +350,6 @@ } } } - } catch (Exception e) { - boolean noCleanIfFailure = options.contains(Option.NoCleanIfFailure); - cleanUpOnFailure(state, failure, noCleanIfFailure); - throw e; - } finally { for (Feature f : features) { callListeners(new FeatureEvent(f, FeatureEvent.EventType.FeatureInstalled, false)); } @@ -362,6 +357,10 @@ installed.put(e.getKey(), e.getValue()); } saveState(); + } catch (Exception e) { + boolean noCleanIfFailure = options.contains(Option.NoCleanIfFailure); + cleanUpOnFailure(state, failure, noCleanIfFailure); + throw e; } }