Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

netty all-in-one review/cleanup #1014

Closed
carrot-garden opened this issue Feb 4, 2013 · 15 comments
Closed

netty all-in-one review/cleanup #1014

carrot-garden opened this issue Feb 4, 2013 · 15 comments
Assignees
Milestone

Comments

@carrot-garden
Copy link

netty all-in-one review/cleanup after #870 #1010

  • rename netty-parent -> netty and netty -> netty-bundle or netty-all?
  • make all-in-one non osgi bundle?
  • keep all-in-one as osgi bundle?
  • remove all-in-one completely?
@trustin
Copy link
Member

trustin commented Feb 4, 2013

  • I don't think we need to rename artifactIds.
  • +1 for making all-in-one non-OSGi
  • I'd like to keep all-in-one. Single JAR dependency is often useful for a beginner.

@carrot-garden
Copy link
Author

-1: current project/module naming can be confusing for a beginner.

@trustin
Copy link
Member

trustin commented Feb 4, 2013

We've been using the artifactId netty for all-in-one JAR since 3. I'm not sure we have to change that now.

@normanmaurer
Copy link
Member

I agree with @trustin . I see no reason for a change

@dantran
Copy link
Contributor

dantran commented Feb 4, 2013

perhaps it is a good time to change it since netty 4 is a huge refactoring comparing to netty-3 anyway. Name it as netty-all looks much more obvious.

@jestan
Copy link
Member

jestan commented Feb 5, 2013

may be netty-all.jar -> netty-uber.jar ?

@trustin
Copy link
Member

trustin commented Feb 5, 2013

Renaming netty to netty-something sounds good to me. (I'd stick to netty-parent though.) Here are the names I thought about:

  • netty-all.jar
  • netty-uber.jar
  • netty-all-in-one.jar
  • netty-all_in_one.jar (because we use '-' like a file separator for module names)
  • netty-☠.jar (Just kidding)

I'm leaning toward netty-all-in-one.jar. Any objections?

@ghost ghost assigned trustin Feb 5, 2013
@dantran
Copy link
Contributor

dantran commented Feb 5, 2013

netty-all.jar, i have seen other package do that convention. netty-all-in-on.jar looks long :-)

@normanmaurer
Copy link
Member

I also vote for netty-all.jar

@trustin
Copy link
Member

trustin commented Feb 5, 2013

Renamed to netty-all.jar.

@normanmaurer
Copy link
Member

@trustin can we close this ?

@carrot-garden
Copy link
Author

+1 rename netty-parent -> netty :-)

@normanmaurer
Copy link
Member

after talking with @trustin we thing we are good here.. Closing it

@carrot-garden
Copy link
Author

this feels like a hack

    <!-- The example depends on (almost) all modules either directly or transitively -->
    <dependency>
      <groupId>${project.groupId}</groupId>
      <artifactId>netty-example</artifactId>
      <version>${project.version}</version>
      <scope>compile</scope>
      <optional>true</optional>
    </dependency>

I would rather have all modules listed explicitly

@carrot-garden carrot-garden reopened this Feb 10, 2013
@normanmaurer
Copy link
Member

Agree

Sent from my iPhone. Excuse any typos....

Am 10.02.2013 um 16:02 schrieb Andrei Pozolotin notifications@github.com:

this feels like a hack

<!-- The example depends on (almost) all modules either directly or transitively -->
<dependency>
  <groupId>${project.groupId}</groupId>
  <artifactId>netty-example</artifactId>
  <version>${project.version}</version>
  <scope>compile</scope>
  <optional>true</optional>
</dependency>

I would rather have all modules listed explicitly


Reply to this email directly or view it on GitHub..

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants