[ checkstyle-Patches-1366035 ] Handle multiple top-level package names in import-control

View: New views
1 Messages — Rating Filter:   Alert me  

[ checkstyle-Patches-1366035 ] Handle multiple top-level package names in import-control

by SourceForge.net :: Rate this Message:

Reply to Author | View Threaded | Show Only this Message

Patches item #1366035, was opened at 2005-11-25 06:06
Message generated for change (Comment added) made by kutzi
You can respond by visiting:
https://sourceforge.net/tracker/?func=detail&atid=397080&aid=1366035&group_id=29721

Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: None
Group: None
Status: Open
Resolution: None
Priority: 5
Private: No
Submitted By: Darrell DeBoer (bigdaz)
Assigned to: Oliver Burn (oburn)
Summary: Handle multiple top-level package names in import-control

Initial Comment:
Our source tree consists of some packages net.foo.* and
others com.foo.*, and we wanted to use the new import
control check to control imports between these packages
as well as on other packages.

The current import-control check assumes a single
top-level package in <import-control pkg="foo"/>. We
have modified the check to be configured like:
<import-control>
   <package name="com.foo">
     ...
   </package>
   <package name="net.foo">
     ...
   </package>
</import-control>

This gives us complete control over imports in our
source tree.

The zip file contains a patch file generated against
CVS on 2005-11-25 and 2 new files serving as testinputs
for the new unit tests.

Hope you find this helpful
ciao
Daz

----------------------------------------------------------------------

Comment By: Christoph Kutzinski (kutzi)
Date: 2009-10-05 11:57

Message:
It would be really great if the patch could be applied as I desparately
need it, too.

----------------------------------------------------------------------

Comment By: Oliver Burn (oburn)
Date: 2009-03-08 06:20

Message:
I am slack and have to applied this patch in an orderly manner. It is
possible that you update the patch to work with SVN head?

----------------------------------------------------------------------

Comment By: Nobody/Anonymous (nobody)
Date: 2009-02-25 15:28

Message:
+ 1 for this one!
Would be great. Thanks.

----------------------------------------------------------------------

Comment By: David DIDIER (c_inconnu)
Date: 2009-01-21 00:59

Message:
Hi, it seems that this patch was not applied ? Is there any chance that it
will be ? Thanks

----------------------------------------------------------------------

Comment By: Oliver Burn (oburn)
Date: 2005-11-28 23:14

Message:
Logged In: YES
user_id=218824

Thanks - I will apply for the 4.1 release.

----------------------------------------------------------------------

You can respond by visiting:
https://sourceforge.net/tracker/?func=detail&atid=397080&aid=1366035&group_id=29721

------------------------------------------------------------------------------
Come build with us! The BlackBerry® Developer Conference in SF, CA
is the only developer event you need to attend this year. Jumpstart your
developing skills, take BlackBerry mobile applications to market and stay
ahead of the curve. Join us from November 9-12, 2009. Register now!
http://p.sf.net/sfu/devconf
_______________________________________________
Checkstyle-devel mailing list
Checkstyle-devel@...
https://lists.sourceforge.net/lists/listinfo/checkstyle-devel