Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 365017 - Initial Code Contribution - SnipMatch Plug-in
Summary: Initial Code Contribution - SnipMatch Plug-in
Status: CLOSED FIXED
Alias: None
Product: z_Archived
Classification: Eclipse Foundation
Component: Recommenders (show other bugs)
Version: unspecified   Edit
Hardware: All All
: P3 normal (vote)
Target Milestone: ---   Edit
Assignee: Marcel Bruch CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-11-28 23:33 EST by Doug Wightman CLA
Modified: 2019-07-24 14:35 EDT (History)
1 user (show)

See Also:


Attachments
source code (115.19 KB, application/zip)
2011-11-28 23:35 EST, Doug Wightman CLA
no flags Details
source code (337.91 KB, application/zip)
2011-12-01 16:25 EST, Doug Wightman CLA
no flags Details

Note You need to log in before you can comment on or make changes to this bug.
Description Doug Wightman CLA 2011-11-28 23:33:46 EST
Build Identifier: 

SnipMatch is a snippet search tool with a shared, central repository. You can learn more and watch a video at: snipmatch.org. We are not open-sourcing this plug-in.

Reproducible: Always
Comment 1 Doug Wightman CLA 2011-11-28 23:35:17 EST
Created attachment 207640 [details]
source code

To the best of my knowledge I hereby state that we - Zi Ye and me, Doug Wightman  -
1. authored 100% of the content we are contributing
2. have the rights to contribute the content to Eclipse
3. are contributing the content under the EPL
Comment 2 Marcel Bruch CLA 2011-11-29 00:55:05 EST
Doug, I guess there is a typo but I have to ask. You write:

"We are not open-sourcing this plug-in." Do you mean "now" instead of "not"?
Comment 3 Doug Wightman CLA 2011-11-29 01:05:29 EST
Correction: I meant to write "we are now open-sourcing this plug-in". :)

(In reply to comment #0)
> Build Identifier: 
> 
> SnipMatch is a snippet search tool with a shared, central repository. You can
> learn more and watch a video at: snipmatch.org. We are not open-sourcing this
> plug-in.
> 
> Reproducible: Always
Comment 4 Doug Wightman CLA 2011-11-29 01:05:51 EST
Yes, sorry about that!

(In reply to comment #2)
> Doug, I guess there is a typo but I have to ask. You write:
> 
> "We are not open-sourcing this plug-in." Do you mean "now" instead of "not"?
Comment 5 Marcel Bruch CLA 2011-11-29 01:10:32 EST
Good to hear :) The attached zip file only contains the server files. I guess the client code also belongs to the contribution?

If so, can you add it to the attached zip file, i.e., replace the attachment with a new, more complete one?
Comment 6 Zi Ye CLA 2011-11-30 17:36:56 EST
To the best of my knowledge I hereby state that we - Doug Wightman and me, Zi Ye -
1. authored 100% of the content we are contributing
2. have the rights to contribute the content to Eclipse
3. are contributing the content under the EPL
Comment 7 Doug Wightman CLA 2011-12-01 16:25:58 EST
Created attachment 207815 [details]
source code
Comment 8 Marcel Bruch CLA 2011-12-01 17:09:57 EST
Hi Doug. Thanks for the updated sources. I initiated/requested the IP check.
Comment 9 Marcel Bruch CLA 2012-02-02 02:02:19 EST
Doug, Zi (probably Zi), can you answer Sharon's questions below to solve the last issues before the contribution get's accepted?

------- Comment #8 From Sharon Corbett 2012-01-23 16:50:40 [reply] -------
Hi Marcel:

I've completed the initial triage and I'll need your help to understand the
following:

1.  SnippetMatchEnvironment.java and MatchEnvironment.java include the
following comment, "must be implemented by derived classes".  I'd like your
help to understand this comment.

2.  SubmitBox.java includes the following, "By checking the box above, you
confirm that you have full rights to release the code in this snippet, and
agree to release it to the public domain. You also understand that       
"Language Interfaces is not responsible for any misuse of this code".   I'd
like your help to understand this comment.

3.  SnipMatchPlugin.java contains, "this feature is only available for
registered users". Is this no longer applicable?

4.  JavaSnippetMatchEnvironment.java references the use of Code Assist.  Is it
correct to understand there is no source contained from Code Assist?

5.  Feature.xml contains copyright to SnipMatch as well as a license URL that
is not to the EPL.  I'm guessing both items are no longer applicable? 


Thanks for your time.

Kind Regards,
Sharon
Comment 10 Doug Wightman CLA 2012-02-02 11:16:25 EST
Thanks Marcel and Sharon, sorry for the delay. Here are our answers:

1. This comment is redundant and can be removed. The "abstract" keyword already makes that clear.

2. We mentioned this in a prior email - it seems to make sense to have snippet contributors indicate that they have the right to share the code that they are publicly sharing, and that we should also (at least initially) only accept code that all our users are then free to use as they wish. This wording should probably be refined by lawyers.

3. Yes, we think it will still make sense to have some features limited to registered users.

4. There is no source taken from code assist. SnipMatch simply performs code assist invisibly on a dummy document, and uses the results.

5. Yes, that is correct.



(In reply to comment #9)
> Doug, Zi (probably Zi), can you answer Sharon's questions below to solve the
> last issues before the contribution get's accepted?
> 
> ------- Comment #8 From Sharon Corbett 2012-01-23 16:50:40 [reply] -------
> Hi Marcel:
> 
> I've completed the initial triage and I'll need your help to understand the
> following:
> 
> 1.  SnippetMatchEnvironment.java and MatchEnvironment.java include the
> following comment, "must be implemented by derived classes".  I'd like your
> help to understand this comment.
> 
> 2.  SubmitBox.java includes the following, "By checking the box above, you
> confirm that you have full rights to release the code in this snippet, and
> agree to release it to the public domain. You also understand that       
> "Language Interfaces is not responsible for any misuse of this code".   I'd
> like your help to understand this comment.
> 
> 3.  SnipMatchPlugin.java contains, "this feature is only available for
> registered users". Is this no longer applicable?
> 
> 4.  JavaSnippetMatchEnvironment.java references the use of Code Assist.  Is it
> correct to understand there is no source contained from Code Assist?
> 
> 5.  Feature.xml contains copyright to SnipMatch as well as a license URL that
> is not to the EPL.  I'm guessing both items are no longer applicable? 
> 
> 
> Thanks for your time.
> 
> Kind Regards,
> Sharon
Comment 12 Marcel Bruch CLA 2012-06-09 15:11:32 EDT
Set target milestone for fixed bugs to 1.0
Comment 13 Marcel Bruch CLA 2012-06-09 15:11:57 EDT
Set target milestone for fixed bugs to 1.0