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

Florian/Dmitry: thanks for the IP confirmations. Since we had build successful, I merged the virgoisation branch into master, deleted the branch, and pushed.

Thank you both for your contributions!

Glyn
On 20 Jul 2010, at 14:29, Glyn Normington wrote:

> "BUILD SUCCESSFUL". Nice work guys!
> 
> Glyn
> On 20 Jul 2010, at 14:19, Glyn Normington wrote:
> 
>> Thanks Dmitry! I've applied those patches to the virgoisation branch and am trying a build.
>> 
>> I won't be able to push until I get Florian's IP confirmation though...
>> 
>> Glyn
>> On 20 Jul 2010, at 14:01, Dmitry Sklyut wrote:
>> 
>> 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<mailto: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><mailto: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><mailto: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><mailto: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><mailto: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<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
>> 
>> <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