Skip to content
This repository has been archived by the owner on Feb 25, 2023. It is now read-only.

Split package akka.http.ccompat #1

Closed
lefou opened this issue Jan 20, 2020 · 1 comment
Closed

Split package akka.http.ccompat #1

lefou opened this issue Jan 20, 2020 · 1 comment

Comments

@lefou
Copy link
Contributor

lefou commented Jan 20, 2020

The two artifacts akka.parsing and akka.http.core contain (disjunct) classes in the Java package akka.http.ccompat. Found in version 10.1.11. A split package is consideres bad practice, even in plain Java world. Split packages are even not possible when working with sealed packages.

We have various options here:

  • Move all classes to the same package into the same artifact
  • Rename one of the packages to avoid the split-package issue, preferably choosing a package name which shares the artifact name as it's prefix
  • With a very good reason (is that even possible?) I think we can keep the split package, but it is the most painful and dirty solution

Related to akka/akka#28304 (comment)

@atooni
Copy link
Member

atooni commented Jun 23, 2020

We are closing the issue as resolving some of the split packages in akka itself might cause binary incompatibilities. We have found another approach to handle split-packages as outlined in the README

@atooni atooni closed this as completed Jun 23, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants