Merging of uberjar resources

[weavejester]

Does this library support merging of uberjar resources, such as data_readers.clj?

[EwenG]

The badigeon.uberjar/bundle function ignores resources conflicts (it does not copy them) and optionally prints a warning when conflicts are found.
The badigeon.uberjar/find-resource-conflicts function returns the path of all the resources that are in conflict.
Then it's up to the user of the library to handle conflicts, typically by doing some file manipulation between a call to badigeon.uberjar/bundle and badigeon.zip/zip.
See also the uberjar example.

[weavejester]

Okay, so there's no automatic conflict resolution like lein uberjar. Is that a planned/wishlist feature, or not on the roadmap?

[EwenG]

That's definitely something that could be implemented. What merging strategy would you like to see implemented ?

[weavejester]

Something configurable would be ideal. Leiningen uses a map of filename strings/patterns to a vector of read, merge and write functions. By default it defines the following:

(def components-merger
  [components-read into components-write])

(def clj-map-merger
  [(comp read-string slurp) merge #(spit %1 (pr-str %2))])

(def concat-merger
  [slurp #(str %1 "\n" %2) spit])

(def default-mergers
  {"META-INF/plexus/components.xml" components-merger
   "data_readers.clj"               clj-map-merger
   #"META-INF/services/.*"          concat-merger}

(See the leiningen.uberjar for the definition of components-read and components-write.)

[EwenG]

There is a first version of badigeon.uberjar/merge-resource-conflicts on master

[weavejester]

Oh thanks! I'll check it out!

[jlesquembre]

I was testing it, and I got an error calling badigeon.uberjar/merge-resource-conflicts:

(err) Execution error (FileNotFoundException) at java.io.FileOutputStream/open0 (FileOutputStream.java:-2).
(err) /.../target/lib-0.0.1/META-INF/services/com.fasterxml.jackson.core.JsonFactory (No such file or directory)

You can reproduce the error if you add cheshire to your dependencies.

I think the problem is here:

badigeon/src/badigeon/uberjar.clj

Lines 267 to 269 in 4bcfd07

	#"META-INF/services/.*" {:read slurp
	:merge #(str %1 "\n" %2)
	:write spit}})

First time, the file doesn't exist yet

I also notice that there is an error in the example file here:

badigeon/sample/badigeon/sample.clj

Line 266 in 4bcfd07

(uberjar/merge-resource-conflicts {:out-path out-path

But the function expects out-path as its first argument:

badigeon/src/badigeon/uberjar.clj

Line 281 in 4bcfd07

([out-path {:keys [deps-map aliases resource-mergers]

[EwenG]

@jlesquembre Thank you, this is fixed on master