> +                                    maven().groupId(KARAF_GROUP_ID)
> +                                            .artifactId(KARAF_ARTIFACT_ID)
> +                                            .versionAsInProject()
> +                                            .type("tar.gz"))
> +                            .karafVersion(getKarafVersion())
> +                            .name("Apache Karaf Distro")
> +                            .unpackDirectory(new 
> File("target/paxexam/unpack/")),
> +                    //We are adding the generated features-repo to the list 
> of known repositories so that we don't have to
> +                    // install artifacts to local maven repo to make the 
> tests work.
> +                    
> editConfigurationFileExtend("etc/org.ops4j.pax.url.mvn.cfg", 
> "org.ops4j.pax.url.mvn.defaultRepositories", ",file:" + new 
> File("../feature/target/features-repo").getCanonicalPath() + "@snapshots")
> +            );
> +        } catch (IOException e) {
> +            Throwables.propagate(e);
> +        } finally {
> +            return option;
> +        }

What does the finally do? Isn't this just
```
try {
  ...
  return option;
} catch (...) {
  Throwables.propagate(e);
}
```
?

---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds-karaf/pull/25/files#r5577774

Reply via email to