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

Hide c.g.inject.internal #16

Closed
gissuebot opened this issue Jul 7, 2014 · 6 comments
Closed

Hide c.g.inject.internal #16

gissuebot opened this issue Jul 7, 2014 · 6 comments

Comments

@gissuebot
Copy link

From crazyboblee on February 14, 2007 18:36:32

So IDEs don't expose the classes to users.

Original issue: http://code.google.com/p/google-guice/issues/detail?id=16

@gissuebot
Copy link
Author

From limpbizkit on February 14, 2007 16:09:14

The .jar for Guice could hide these classes from OSGi-compliant tools like Eclipse.
Here's a sample Ant task that creates the necessary meta information for the .jar file:

  <taskdef name="osgiBundleInfo"
classname="org.knopflerfish.ant.taskdefs.bundle.BundleInfoTask"
classpath="tools/OSGiBundleInfo.jar"/>
  <osgiBundleInfo exports="packagelist">
     <fileset dir="${basedir}">
        <include name="/source//.java"/>
        <exclude name="/util//
.java"/>
     </fileset>
  </osgiBundleInfo>

The task can be downloaded here: http://www.knopflerfish.org/

@gissuebot
Copy link
Author

From kevinb9n on February 28, 2007 11:06:43

(No comment was entered for this change.)

Labels: -Priority-Medium Priority-Low

@gissuebot
Copy link
Author

From kevinb9n on May 31, 2007 13:35:53

(No comment was entered for this change.)

Summary: Hide c.g.inject.internal

@gissuebot
Copy link
Author

From sberlin on October 26, 2010 07:35:45

Final iteration of the patch that makes it work flawlessly for asm, cglib & internal.util, regardless of the number of subpackages within each.

The patch adds rules such that all jarjar'd classes get their classname prefixed with $, so that IDEs do not suggest the classes as possible import completions.

Attachment: gist
   hide-jarjar-and-internal.util-once-and-for-all.txt

@gissuebot
Copy link
Author

From limpbizkit on October 26, 2010 09:41:44

Fantastic.

@gissuebot
Copy link
Author

From sberlin on October 26, 2010 18:49:19

fixed (or framework for hiding other things, atleast) in r1322 .

Status: Fixed

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

No branches or pull requests

1 participant