You are viewing a plain text version of this content. The canonical link for it is here.
Posted to mime4j-dev@james.apache.org by Wim Jongman <wi...@gmail.com> on 2009/09/07 11:02:47 UTC

Re: newsreader contribution

*Greetings, Mime4j developers. We are planning to use mime4j into an Eclipse
project. I will elaborate more with a separate e-mail to the list. *

Hi Scott,
1. Glad to hear the good news. Mime4j is needed to disect the messages and i
don't know of any java.* package that does this nor is there currently an
Orbit package that does this. I suspect that apache code will pass quite
easy. I will start the process of getting the library into Orbit.

2. Naming it Salvo was the plan, yes. Just like Tweethub is a better name
then *ecf.twitter* However, I agree that we have to move the core protocol
into org.eclipse.ecf.nntp but I would like to keep Salvo as the name for the
product.

3. Copyright, Will do that. Is there a tool to do stuff like that?

4. Execution environment. I will try to move the model and the core down.

Sounds good, what is the estimated time for moving it from OSUOSL to
Eclipse? what will be the criterium for the move?

Best regards,

Wim


On Mon, Sep 7, 2009 at 1:56 AM, Scott Lewis <sl...@eclipsesource.com>wrote:

> Hi Wim,
>
>
> In short, things look great...again many thanks for the contribution to ECF
> of the work.
>
> I have a couple of suggestions to make the IP review process
> easier/quicker, and I'll enumerate them below, so that we/I don't repeat
> ourselves as additional more specific reviews are done by me and/or other
> ECF committers.
> 1) Plugin structure looks great to me (separation between core, model, ui,
> and application is good).  I noticed that the Apache mime4j is included as a
> dependency (for salvo.ui plugin) and since this is obviously an Apache
> library, and as far as I can tell it is *not* currently present in Orbit,
> we'll probably have to submit an IQ for this library separately...and
> eventually perhaps contribute it to the Orbit project.  Is it possible that
> the salvo.ui plugin could work without this dependency?...(or perhaps on
> some other existing library/bundle from Orbit or other EF project).
>
> 2) The existing bundles/packages are named com.weltevree.salvo.*.  Do you
> want to have the name of the project be 'salvo'?  Eventually, we'll need to
> decide on some project/bundle/package naming in the org.eclipse.ecf.*
> namespace.  I would suggest something like org.eclipse.ecf.nntp.*, or
> perhaps org.eclipse.ecf.news.*...this is something we should probably
> jointly decide.
>
> 3) Every source file will eventually need a copyright notice.  Below (as an
> example), are the copyright notices that I use for EclipseSource or
> Composent (my own company).  You can use these as examples if you wish, and
> either put a personal (i.e. just you) in the copyright, or co-authors, or an
> organization/company...as is appropriate (you as the author of the code must
> be able describe the provenance of all of the source for the IP review).
>
> 4) I notice that the execution environment for most of the plugins was set
> to J2SE-1.5.  This is *fine*, but *if possible*, it would be great to move
> this *down* to J2SE-1.4 or even CDC 1.1/Foundation 1.1.  This would allow
> the relevant code to run on smaller/older runtimes.  For example, perhaps
> the model bundle could be moved down to CDC 1.1/Foundation 1.1
>
> When you are ready, I can/will create an account for you on the ECF OSU
> Open Source Lab CVS server (ecf1.osuosl.org) and we can make the projects
> available on that CVS server, and at that point ECF committers will be able
> to help with the editing/etc prior to going through the CQ/IP process.  This
> is where we have the TwitterHub application for example.
>
> In the longer term, I see some great opportunities to take the newsreader
> model and make it into a new ECF API (or perhaps merge it with the existing
> bulletin board API).  Given ECF's structure (of having container adapters
> that expose particular protocols), I think this should be natural to do, and
> I and/or other ECF committers can/will help with this.  This would allow
> easy consumption of the newsreader model within other applications and/or
> UIs.  It would/will also make integration with existing ECF apps (like
> TweetHub) much more easy/natural.
>
> Thanks,
>
> Scott
>
>
> /****************************************************************************
> * Copyright (c) 2004, 2009 Composent, Inc. and others.
> * All rights reserved. This program and the accompanying materials
> * are made available under the terms of the Eclipse Public License v1.0
> * which accompanies this distribution, and is available at
> * http://www.eclipse.org/legal/epl-v10.html
> *
> * Contributors:
> *    Composent, Inc. - initial API and implementation
>
> *****************************************************************************/
>
>
> /*******************************************************************************
> * Copyright (c) 2009 EclipseSource and others. All
> * rights reserved. This program and the accompanying materials are made
> * available under the terms of the Eclipse Public License v1.0 which
> * accompanies this distribution, and is available at
> * http://www.eclipse.org/legal/epl-v10.html
> *
> * Contributors: Scott Lewis - initial API and implementation
>
> *******************************************************************************/
>
> Wim Jongman wrote:
>
>> Hi,
>>
>> I am currently writing a newsreader client which I would like to
>> contribute to ECF. If you want to help us review the code to get this going,
>> please sent me a mail and I will sent you svn coordinates. See screeenshot
>> and wiki.eclipse.org/newsreader <http://wiki.eclipse.org/newsreader>
>>
>> Best regards,
>>
>> Wim
>>
>>
>>
>>
>>
>> ------------------------------------------------------------------------
>>
>>
>> ------------------------------------------------------------------------
>>
>> _______________________________________________
>> ecf-dev mailing list
>> ecf-dev@eclipse.org
>> https://dev.eclipse.org/mailman/listinfo/ecf-dev
>>
>>
>
>