Last Comment Bug 411958 - Use tinderboxen to build gdata-provider nightlies
: Use tinderboxen to build gdata-provider nightlies
Status: VERIFIED FIXED
[gdata-0.4]
:
Product: Calendar
Classification: Client Software
Component: Build Config (show other bugs)
: unspecified
: All All
: -- normal (vote)
: 0.9
Assigned To: Philipp Kewisch [:Fallen]
:
Mentors:
: 419260 (view as bug list)
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2008-01-11 11:31 PST by Philipp Kewisch [:Fallen]
Modified: 2008-06-20 09:12 PDT (History)
2 users (show)
See Also:
Crash Signature:
QA Whiteboard:
Iteration: ---
Points: ---


Attachments
Enable building and uploading gdata (5.24 KB, patch)
2008-01-16 12:48 PST, Philipp Kewisch [:Fallen]
ause: review+
Details | Diff | Review
All Tinderbox configs as checked in (5.38 KB, patch)
2008-03-18 05:22 PDT, Philipp Kewisch [:Fallen]
no flags Details | Diff | Review
Mac fixes - v1 (6.42 KB, patch)
2008-04-02 13:03 PDT, Philipp Kewisch [:Fallen]
ause: review+
Details | Diff | Review

Description Philipp Kewisch [:Fallen] 2008-01-11 11:31:10 PST
Thanks to bug 410922, it is now possible to build the gdata provider and upload it as an extension to ftp.mozilla.org.

It is sufficient to let one tinderbox build the gdata provider, it might look strange if only one tinderbox creates this xpi. Depending on how latest-mozilla1.8 is linked, we might want to go ahead an let all tinderboxen build the xpi.
Comment 1 Philipp Kewisch [:Fallen] 2008-01-16 12:48:35 PST
Created attachment 297403 [details] [diff] [review]
Enable building and uploading gdata

Enable build for gdata. For now I just picked the sunbird linux tinderbox to build gdata. I would land the Makefile and install.rdf on both branches, test the nightly creation on one tinderbox and see if it uploads to the correct location.

The way it is with this patch, I believe it will land in:

ftp://ftp.mozilla.org/pub/calendar/sunbird/nightly/latest-mozilla1.8/xpi

Is this path ok? Setting up a second tinderbox with the same configs might overwrite the same xpi file, so probably only one is needed. The same change could be done on a trunk tinderbox, then it will also be in latest-trunk/xpi.

Thinking about alternatives, I don't think I can land it in pub/calendar/xpi so easily.
Comment 2 Stefan Sitter 2008-01-16 12:58:45 PST
Comment on attachment 297403 [details] [diff] [review]
Enable building and uploading gdata

>+++ calendar/providers/gdata/install.rdf	16 Jan 2008 20:22:16 -0000
><!-- Thunderbird -->
><em:id>{3550f703-e582-4d05-9a08-453d09bdfdc6}</em:id>
><em:minVersion>2.0a1</em:minVersion>
><em:maxVersion>3.0a1</em:maxVersion>

I think the extension should only install in Thunderbird as supported by the required Lightning version to avoid confusion. For example the gdata extension build that requires 0.8pre should only install in Thunderbird 2.0.0.0 to 2.0.0.*.
Comment 3 Philipp Kewisch [:Fallen] 2008-01-17 00:05:54 PST
I didn't think of thunderbird versioning, but I think we are fine (for now). In case we don't check out a mail tree we won't have the version numbers. Also, I believe the gdata provider also runs on thunderbird trunk without problems. The knockout criterium on thunderbird trunk will be that the <em:requires> will require 0.8pre, which will not run on tb trunk.

Is this sufficient, or should I do the following:

>#ifdef MOZILLA_1_8_BRANCH
><em:minVersion>2.0a1</em:minVersion>
><em:maxVersion>2.0.0.*</em:maxVersion>
>#else
><em:minVersion>3.0a1</em:minVersion>
><em:maxVersion>3.0.0.*</em:maxVersion>
>#endif
Comment 4 Philipp Kewisch [:Fallen] 2008-03-01 11:51:43 PST
*** Bug 419260 has been marked as a duplicate of this bug. ***
Comment 5 Philipp Kewisch [:Fallen] 2008-03-14 14:57:35 PDT
ause, do you mind reviewing this soon, so I can more easily create nightlies?
Comment 6 ause 2008-03-17 11:29:31 PDT
Comment on attachment 297403 [details] [diff] [review]
Enable building and uploading gdata

activation in all lightning tb looks best to me
Comment 7 Philipp Kewisch [:Fallen] 2008-03-17 12:52:38 PDT
I want to check this in tomorrow. Daniel, is that ok with you also for SUNBIRD_0_8_BRANCH ?
Comment 8 Daniel Boelzle [:dbo] 2008-03-18 02:17:48 PDT
Comment on attachment 297403 [details] [diff] [review]
Enable building and uploading gdata

>+DIRS = base \
>+       caldav \
>+       composite \
>+       gdata \
>+       ics \
>+       memory \
>+       storage \
>+       wcap \
Please add $(NULL).
Comment 9 Daniel Boelzle [:dbo] 2008-03-18 02:20:46 PDT
Ok for 0.8.
Comment 10 Philipp Kewisch [:Fallen] 2008-03-18 05:22:59 PDT
Created attachment 310237 [details] [diff] [review]
All Tinderbox configs as checked in

Fixed $(NULL). These are the tinderbox configs I'm checking in, for lightning.
Comment 11 Philipp Kewisch [:Fallen] 2008-03-18 05:35:20 PDT
Gdata files checked in on HEAD and MOZILLA_1_8_BRANCH and SUNBIRD_0_8_BRANCH.
tinderbox changes win/linux/mac checked in on HEAD and MOZILLA_1_8_BRANCH
tinderbox changes solaris-* checked in on MOZILLA_1_8_BRANCH

-> FIXED
Comment 12 Philipp Kewisch [:Fallen] 2008-03-18 06:46:47 PDT
Not working for mac: The tinderbox is looking for the gdata xpi from the lightning-obj/ppc/dist/universal/xpi-stage/ directory, but gdata is in the non-universal directory.
Comment 13 Stefan Sitter 2008-03-18 07:41:25 PDT
Seems the correct Thunderbird Trunk / 1.8 Branch support mentioned in Comment #2 and Comment #3 is missing from the checkin?
Comment 14 ause 2008-03-18 10:47:44 PDT
for mac problem see mac mozconfig of lightning tb and calendar/lightning/build/universal.mk
Comment 15 Philipp Kewisch [:Fallen] 2008-04-02 13:01:13 PDT
(In reply to comment #13)
> Seems the correct Thunderbird Trunk / 1.8 Branch support mentioned in Comment
> #2 and Comment #3 is missing from the checkin?
> 

Since I had no reply regarding my question there, I decided to leave it out. Reopening to fix the mac issue, and since I don't really mind, I'll also put in the trunk version change.
Comment 16 Philipp Kewisch [:Fallen] 2008-04-02 13:03:47 PDT
Created attachment 313156 [details] [diff] [review]
Mac fixes - v1

Untested, but something like this should fix it. ause, it would be great if you could give this patch a test on a working build mac.
Comment 17 ause 2008-04-03 02:56:27 PDT
daniel promised to do the testing.
Comment 18 Daniel Boelzle [:dbo] 2008-04-03 13:44:00 PDT
my universal build has both xpis:

  /Users/dbo/moz18/sbird-uni-opt-Darwin/ppc/dist/universal/xpi-stage:
  total used in directory 2472 available 16459660
  drwxr-xr-x  5 dbo  staff      170 Apr  3 18:22 .
  drwxr-xr-x  4 dbo  staff      136 Apr  3 17:40 ..
  -rw-r--r--  1 dbo  staff    44847 Apr  3 18:22 gdata-provider.xpi
  drwxr-xr-x  9 dbo  staff      306 Apr  3 18:22 lightning
  -rw-r--r--  1 dbo  staff  1219854 Apr  3 18:22 lightning.xpi
Comment 19 Philipp Kewisch [:Fallen] 2008-04-09 10:51:27 PDT
Checked in on HEAD and MOZILLA_1_8_BRANCH

-> FIXED

Note You need to log in before you can comment on or make changes to this bug.


Privacy Policy