[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index] [List Home]
Re: [virgo-dev] Re: snaps checked in

Here is the follow up: https://bugs.eclipse.org/bugs/show_bug.cgi?id=320387

Regards,

Dmitry

On Tue, Jul 20, 2010 at 8:35 AM, Florian Waibel <fwaibel@xxxxxxxxxxxxxxxxx> wrote:
I created the bug as requested: https://bugs.eclipse.org/bugs/show_bug.cgi?id=320382

Regards,
 florian


> To keep the IP process simple, it would be great if Florian could raise a bugzilla and attach "0001 patch" since that is his work. Florian, is that possible?
>
> In view of the due diligence process [1], both you and Florian should confirm in the bugzillas that you wrote 100% of the code you contributed, that you have the right to contribute the code to Eclipse, and that any new Java files contain the appropriate License header.
>
> Thanks,
> Glyn
> [1] http://www.eclipse.org/legal/EclipseLegalProcessPoster.pdf
>
> On 16 Jul 2010, at 19:40, Dmitry Sklyut wrote:
>
> I would like to follow Florian's example.
>
> Attached are a set of patches to get Snaps building against 2.1.0.M02-incubation of virgo and 1.1.0.M02-incubation of gemini-web.
>
> I got it to the point that simple-host + spring-snap get deployed and bound.  From my experience with Slices - there are still 2/3 issues that need to be fixed before it is usable.  I will look into creating patches for those once this set is settled.
>
> To test:
>
> apply patches.
> cd build-snap
> ant clean clean-integration test package
> cp target/package-expanded/snaps-1.0.0.BUILD-${localbuildtime}/repository/usr/* $virgo_home/repository/usr
> modify $virgo_home/config/org.eclipse.virgo.kernel.userregion.properties to add snaps to the list of initialArtifacts (not sure if this is necessary, maybe dropping a plan in pickup is enough)
> Example:
> initialArtifacts=repository:plan/org.eclipse.virgo.kernel.userregion.springdm, repository:plan/org.eclipse.virgo.web, repository:plan/org.eclipse.virgo.snaps
>
> Start webserver.
>
> cp $snaps/org.eclipse.virgo.snaps.test/src/test/resources/simple-host.war  to $virgo-home/pickup/
> Test:
> http://localhost:8080/simple-host
>
> cp $snaps/org.eclipse.virgo.snaps.test/src/test/resources/spring-snap.jarr  to $virgo-home/pickup/
> Test:
> http://localhost:8080/simple-host/spring/s/withViewName
> http://localhost:8080/simple-host/spring/s/write
>
>
> Note on patches:
> 0005 and 0006 should have been single - but I forgot to git add . before committing
> 0001 patch is basically application of everything that Florian attached to this email thread.
>
> I also bumped version of virgo-build to align with other virgo components (took version from virgo.web one - hope it is right).
>
> Let me know if I should create a Bugzilla issue to track this.
>
> Regards,
>
> Dmitry
>
>
> On Fri, Jul 9, 2010 at 11:32 AM, Florian Waibel <fwaibel@xxxxxxxxxxxxxxxxx<mailto:fwaibel@xxxxxxxxxxxxxxxxx>> wrote:
> Hi,
>
> this time i tried a little harder.
> But didn't get past a Tests failed (crashed) in the snaps.test project.
> I guess i missed something essential.
>
> In case someone else is trying to migrate snaps i added the updated virgoise.sh.
>
> I'll investigate the problem next week. If someone else could have a closer look, too that would be a great
> timesaver i guess...
>
> Thanks,
> florian
>
>
>
> On Jul 7, 2010, at 3:43 PM, Glyn Normington wrote:
>
>> Hi Florian
>>
>> Thanks very much for spotting that "slice" which slipped through the net! I've fixed that ([1] with an acknowledgement to yourself) and another that showed up via:
>>
>> ls -lR | grep slice
>>
>> The invitation to get the slices code working on top of Virgo instead of dm Server is still open. :-)
>>
>> Glyn
>> [1] http://git.eclipse.org/c/virgo/org.eclipse.virgo.snaps.git/commit/?id=b3494fc78bf727f45c55d4e25f8de335918b6490
>> On 7 Jul 2010, at 13:05, Florian Waibel wrote:
>>
>>> Hi,
>>>
>>> i followed your invitation :)
>>>
>>> #!/usr/bin/env bash
>>> dir=`dirname $0`
>>> dir=`cd $dir;pwd`
>>>
>>> for f in $(find . -iname "*.java" -or -iname "*.aj" -or -iname "*.xml" -or -iname ".project" -or -iname "*.MF" -or -iname "*.mf" -or -iname ".prefs"  -or -iname ".classpath" -or -iname "build.versions" -or -iname "*.properties" -or -iname "*.bat" -or -iname "*.sh" -or -iname "*.conf" -or -iname "*.profile" -or -iname "*.config" -or -iname "*.plan" -or -iname "runner.bundles")
>>> do
>>>
>>> sed -i -e 's/SLICES_IVY_CACHE/SNAPS_IVY_CACHE/g' $f
>>> sed -i -e 's/SLICE_/SNAP_/g' $f
>>> sed -i -e 's/SLICES_/SNAPS_/g' $f
>>> sed -i -e 's/slice\.simple/snap\.simple/g' $f
>>>
>>> rm $f-e
>>> done
>>>
>>> One "missed" refactoring in project test-apps/simple-snap:
>>>
>>> #mkdir -p test-apps/simple-snap/src/main/java/snap/simple
>>> #mv test-apps/simple-snap/src/main/java/slice/simple/*java test-apps/simple-snap/src/main/java/snap/simple
>>> #rmdir test-apps/simple-snap/src/main/slice/simple
>>>
>>> I don't know about the $dir/update-dependency.rb commands. So those lines are missing...
>>>
>>> Regards,
>>> florian
>>>
>>> --
>>> Florian Waibel
>>> EclipseSource
>>> Email: fwaibel@xxxxxxxxxxxxxxxxx<mailto:fwaibel@xxxxxxxxxxxxxxxxx>
>>> Tel: +49-721-66-47-33-0
>>> Fax: +49-721-66-47-33-29
>>> http://www.eclipsesource.com/
>>> ========================= Legal Disclaimer =================================
>>> According to Section 80 of the German Corporation Act
>>> Innoopract Informationssysteme GmbH must indicate the following information:
>>> Address: Stephanienstrasse 20, 76133 Karlsruhe Germany
>>> General Manager: Jochen Krause
>>> Registered Office: Commercial Register Mannheim HRB 107883
>>> ============================================================================
>>>
>>> On Jul 7, 2010, at 9:53 AM, Glyn Normington wrote:
>>>
>>>> Oh, I should also have mentioned the slices->snaps migration notes:
>>>>
>>>> http://wiki.eclipse.org/Virgo/Community#Migrating_from_the_dm_Server_slices_prototype_to_the_Virgo_snaps_prototype
>>>>
>>>> Glyn
>>>> On 6 Jul 2010, at 19:02, Glyn Normington wrote:
>>>>
>>>>> I've checked in snaps to Eclipse git. See [1] for the location.
>>>>>
>>>>> The only changes I've made compared to the slices prototype is to uniformly rename "slices" to "snaps" (including the singular and other capitalisations), to re-package the code to org.eclipse.virgo packages, and to get this building cleanly with Virgo build. It still has dm Server pre-reqs and needs "virgoising" (the virgoise.sh shell script in the web server repo provides a good starting point).
>>>>>
>>>>> If anyone would care to provide a patch to get this working with Virgo, please be my guest.
>>>>>
>>>>> Glyn
>>>>>
>>>>> [1] http://wiki.eclipse.org/Virgo/Source#Virgo_git_Repositories
>>>>
>>>> _______________________________________________
>>>> virgo-dev mailing list
>>>> virgo-dev@xxxxxxxxxxx<mailto:virgo-dev@xxxxxxxxxxx>
>>>> https://dev.eclipse.org/mailman/listinfo/virgo-dev
>>>
>>
>
>
> _______________________________________________
> virgo-dev mailing list
> virgo-dev@xxxxxxxxxxx<mailto:virgo-dev@xxxxxxxxxxx>
> https://dev.eclipse.org/mailman/listinfo/virgo-dev
>
>
> <0001-commiting-changes-from-Florian-Waibel-patches-sent-t.patch><0002-Update-versions-of-dependencies-and-virgo-build-vers.patch><0003-update-virgo-build-to-the-same-version-as-virgo.web.patch><0004-align-version-of-virgo-build-with-other-virgo-compon.patch><0005-upgrade-snaps-to-use-2.1.0-integration-dependencies.patch><0006-upgrade-snaps-to-use-2.1.0-integration-dependencies.patch><0007-change-log-even-prefix-to-match-messages.patch><ATT00001..txt>
>
> _______________________________________________
> virgo-dev mailing list
> virgo-dev@xxxxxxxxxxx
> https://dev.eclipse.org/mailman/listinfo/virgo-dev

_______________________________________________
virgo-dev mailing list
virgo-dev@xxxxxxxxxxx
https://dev.eclipse.org/mailman/listinfo/virgo-dev