The unittests discovered that we don't have the correct license header
for the newest addition to our core project in wicket-1.x branch:
WildcardMatcherHelper.java
2 points:
1. I thought we had our default new file template in place, why doesn't it work?
2. Unit tests are to be run. If a test
that's one of the strangest requirements if you ask me.
Why oh why do you want to force that it needs to be a binary inclusion only?
Thats very strange for an opensource thing...
But rewriting that.. its a file with almost only static finals.. rewriting
that means that you pretty much type it
But rewriting that.. its a file with almost only static finals.. rewriting
that means that you pretty much type it over
What we could do, and what imo would be a bit nicer, is instead of a
general bucket of client properties (backed by a map), just implement
those properties as actual
There is an issue report for it now, so in time if noone has done anything
about it I will. But at the moment I'm trying to get this license header
fun task done. So anyone please submit patches for WICKET-87 if you feel
like it :)
Frank
On 11/18/06, Eelco Hillenius [EMAIL PROTECTED] wrote:
ClientProperties is MPL, which should be ok, right?
Eelco
On 11/16/06, Frank Bille [EMAIL PROTECTED] wrote:
All,
I was a little too quick placing ASL2 headers in all the .java files in 2.0.
When looking trough 1.x I found some thirdparty code. I have therefore
looked through all java files
On 11/17/06, Eelco Hillenius [EMAIL PROTECTED] wrote:
ClientProperties is MPL, which should be ok, right?
If I understand [1] correctly then no. We must only have binary inclusion.
So this actually means that we will have to either use it as a library
(include echo as dependency, yeah
You're right. I was confused with BSD. Damn. Seems like we have to
rewrite it then. To the upside, this isn't very difficult.
Eelco
On 11/17/06, Frank Bille [EMAIL PROTECTED] wrote:
On 11/17/06, Eelco Hillenius [EMAIL PROTECTED] wrote:
ClientProperties is MPL, which should be ok, right?
http://issues.apache.org/jira/browse/WICKET-87
On 11/17/06, Frank Bille [EMAIL PROTECTED] wrote:
I'll add an issue and hope someone volunteers :)
Frank
On 11/17/06, Eelco Hillenius [EMAIL PROTECTED] wrote:
You're right. I was confused with BSD. Damn. Seems like we have to
rewrite it
All,
I was a little too quick placing ASL2 headers in all the .java files in 2.0.
When looking trough 1.x I found some thirdparty code. I have therefore
looked through all java files which doesn't have a ASL2 header but something
else and the following came up:
On 11/16/06, Martijn Dashorst [EMAIL PROTECTED] wrote:
These are public domain, taken from Doug Lea's concurrent utils. Sun
has adopted these for Java 1.5. So for 1.x they need to be there, 2.0
they could be removed in favor of the JDK provided
java.util.concurrent collections.
I don't know
just noticed the below
this is a bad joke right?
markup: 1line
license: 14 lines
are we stripping the license block somewhere? or are we sending it to the
client?
you include two ajax editable choices and you get two of these headers in
the returned markup?
-igor
Modified:
Hmm Will you get it returned? It's a panel so it's only the content of it
that gets returned, right?
Frank
On 11/13/06, Igor Vaynberg [EMAIL PROTECTED] wrote:
just noticed the below
this is a bad joke right?
markup: 1line
license: 14 lines
are we stripping the license block somewhere? or
But you are right though that we still have that issue with were and if we
strip headers. All pages .html (not extended), .css and .js files *will*
have the license header and until we come to an agreement between us and ASF
they stay. And that sucks IMO
Frank
On 11/13/06, Igor Vaynberg [EMAIL
And it increases memory usage. We might not transfer it to the client
but we cache the markup.
Juergen
On 11/13/06, Frank Bille [EMAIL PROTECTED] wrote:
But you are right though that we still have that issue with were and if we
strip headers. All pages .html (not extended), .css and .js files
On 11/13/06, Juergen Donnerstag [EMAIL PROTECTED] wrote:
And it increases memory usage. We might not transfer it to the client
but we cache the markup.
Yes thats right. That sounds like stripping license headers (or replacing
with small one-line notices) at build time is the right solution.
Donnerstag [EMAIL PROTECTED] wrote:
And it increases memory usage. We might not transfer it to the client
but we cache the markup.
Yes thats right. That sounds like stripping license headers (or replacing
with small one-line notices) at build time is the right solution.
i don't hope that #Id# is mandatory!
I hate those things. Because those things mess up merging of branches
because they constantly change.
It is totally stupid that this is the case. A merge should ignore those
completely.
johan
On 11/6/06, Erik van Oosten [EMAIL PROTECTED] wrote:
Hi
My point is that in checkstyle you are free to enforce both the presence
_and absence_ of anything that can be expressed as a regular expression.
So that includes the $Id$ tag.
That I always enforce inclusion of an $Id$ is just an example. In most
of my projects I don't have a lot of merging
to be
in there. I don't expect ASF to relax their stance on those files.
+1 on removing the svn keywords.
Martijn
On 11/5/06, Frank Bille [EMAIL PROTECTED] wrote:
Last night when I couldn't sleep I got this crazy idea to write a unittest
for checking the source files in the project for correct license
But yeah I forgot the .js files. I'll add them right away.
The bad thing about having it as a unittest is that it has to be copied to
every subproject and can't just be run off wicket-parent.
What to do with all these license headers when building binaries/sending
output to client/etc
I don't know checkstyle very well, but doesn't it only handle java code?
Frank
On 11/5/06, Martijn Dashorst [EMAIL PROTECTED] wrote:
On 11/5/06, Frank Bille [EMAIL PROTECTED] wrote:
The bad thing about having it as a unittest is that it has to be copied
to
every subproject and can't just
Their site is now out of order, so I couldn't check it. I thought you
could supply a file pattern.
Anyhow, the check could also be a unit test, and the actual
checking/file scanner could be implemented in Wicket core (though it
is not a specific framework concern imo).
All sub projects could
Hi Martijn,
There are checkstyle plugins for Eclipse and for IDEA. Don't know about
Netbeans.
In addition checkstyle is able to check for a header (even as a RE), or
check for the presence/absence of any RE.
I used checkstyle frequently to enforce a correct copyright header, the
presence of
go ahead.
Is there a tool that can quick fix everything?
johan
On 10/3/06, Martijn Dashorst [EMAIL PROTECTED] wrote:
I ran the RAT tool from Apache on our distribution for 1.2.2, and it
came up with the following files that don't have an ASL header. I
haven't looked into it closely but I
24 matches
Mail list logo