290747 - use sunbird.jar for sunbird specific files

Status: VERIFIED FIXED

(Calendar :: Build Config, defect)

Description

[Michiel van Leeuwen (email: mvl+moz@)]

The current way of everwriting files in calendar.jar for sunbird is painful if
you rebuild, and you forget to rebuild in sunbird.
I suggest to use sunbird.jar for sunbird specific files.

Comment 1

[Michiel van Leeuwen (email: mvl+moz@)]

Attachment: work in progress v1

patch creates sunbird.jar. so far seems to work.
I need to firust out theming.
Also, the commandlineHandling needs to be moved and changed. Still working on
that.

Comment 2

[Michiel van Leeuwen (email: mvl+moz@)]

*** Bug 292771 has been marked as a duplicate of this bug. ***

Comment 3

[Matthew (lilmatt) Willis]

This should be significantly easier with the dropping of the XPI, the move of locales to /l10n, and all the cleanup.

nominating for 0.3

mvl: I'm willing to take this over if you'd like

Comment 4

[Joey Minta]

Nice to have, but we wouldn't hold a release for this.

Comment 5

[Michiel van Leeuwen (email: mvl+moz@)]

-> lilmatt

Comment 6

[Matthew (lilmatt) Willis]

taking

Comment 7

[Gary Kwong [:gkw] [:nth10sd] (NOT official MoCo now)]

mvl, could you please provide an updated status on your patch/bug?

Comment 8

[Michiel van Leeuwen (email: mvl+moz@)]

Nothing really changed since I posted the patch (which is now hopelessly bitrotted). So there is no status to give an update to.

Comment 9

[Philipp Kewisch [:Fallen] [:📆][:🧩]]

Attachment: Step 1 - Move around files

I'd like to do this in two steps to not break too much at once. This step takes care of the following:

* Most jar manifests:
  - Align bracketed values at column 60
  - Replace #expand with #filter substitution and @THEME@
  - get rid of classic/ part, I believe this was done for historic reasons
* Move sunbird/themes/@THEME@/sunbird/ to sunbird/base/themes/@THEME@/
  - Get rid of unneeded Makefile.in's, LICENSE file and contents.rdf's
  - Generalize jar.mn's from sunbird/themes/@THEME@/sunbird/jar.mn to
    sunbird/base/jar.mn
* Move aboutDialog.* to chrome://sunbird/ 
  - adds locales/en-US/chrome/sunbird
  - adds sunbird.jar and sunbird-@AB_CD@.jar
* Get rid of Throbber.png/gif and synch_animated.gif
  - replace winstripe throbber with toolkit throbber images
* Fit packages-static for new jar files

Comment 10

[Philipp Kewisch [:Fallen] [:📆][:🧩]]

Comment on attachment 342786 [details] [diff] [review]
Step 1 - Move around files

Stefan, it would be nice if you could also take a look since you know a lot about jar files and packaging.

Anyone else is also welcome to doublecheck

Comment 11

[Philipp Kewisch [:Fallen] [:📆][:🧩]]

Attachment: Step 2 - Move entities and rules around

This step takes care of moving parts of files out into their sunbird counterpart. Its not necessarily complete (i.e I could move out a bunch of lightning-specific strings into files only packaged with lightning), but it takes care of the most obvious things:

* Move sb specific toolbar css rules into chrome://sunbird/skin/toolbar.css
* Move sb specific entities from calendar.dtd into sunbird.dtd 
  - I've changed the entity names here from calendar.... to sunbird....
* Move sb specific entities from calendar/locale/menuOverlay.dtd into
  sunbird/locale/menuOverlay.dtd
* More jar.mn aligning I forgot to put in Step 1

Step 2 obviously requires Step 1 to be applied (thank you, mq!!)

Comment 12

[Philipp Kewisch [:Fallen] [:📆][:🧩]]

Fixing this bug makes packaging easier since the UI for thunderbird is going in a different direction than the current UI for Sunbird is.

Comment 13

[Philipp Kewisch [:Fallen] [:📆][:🧩]]

Note this bug calls for patches for domi and venkman to fit with the new main window uri for Sunbird. I'll take care when these are checked in and don't cause problems.

Comment 14

[Stefan Sitter [:ssitter]]

Comment on attachment 342786 [details] [diff] [review]
Step 1 - Move around files

From a very, very quick look at the patch:

+ skin/calendar/timezone_10h30.png  (/calendar/prototypes/themes/@THEME@/timezone_10h.png)
+ skin/calendar/timezone_10h.png    (/calendar/prototypes/themes/@THEME@/timezone_10h.png)
+ skin/calendar/timezone_11h30.png  (/calendar/prototypes/themes/@THEME@/timezone_11h.png)
+ skin/calendar/timezone_11h.png    (/calendar/prototypes/themes/@THEME@/timezone_11h.png)
+ skin/calendar/timezone_12h45.png  (/calendar/prototypes/themes/@THEME@/timezone_12h.png)
+ skin/calendar/timezone_12h.png    (/calendar/prototypes/themes/@THEME@/timezone_12h.png)
+ skin/calendar/timezone_13h.png    (/calendar/prototypes/themes/@THEME@/timezone_13h.png)
+ skin/calendar/timezone_14h.png    (/calendar/prototypes/themes/@THEME@/timezone_13h.png)

Wrong images used

+DEFINES += \
+    -DAPP_VERSION="$(APP_VERSION)" \
+	-DTHEME=$(THEME) \
+	$(NULL)

Tabs vs. spaces?

Comment 15

[Philipp Kewisch [:Fallen] [:📆][:🧩]]

(In reply to comment #14)
...
> + skin/calendar/timezone_13h.png   
> (/calendar/prototypes/themes/@THEME@/timezone_13h.png)
> + skin/calendar/timezone_14h.png   
> (/calendar/prototypes/themes/@THEME@/timezone_13h.png)
> 
> Wrong images used

I see...strangely enough this was already that way before. Christian, any ideas?

Took care of the tabs issue, thanks.

Comment 16

[Berend Cornelius]

Comment on attachment 342793 [details] [diff] [review]
Step 2 - Move entities and rules around

Tested both applications and found it was Ok.
r=berend

Comment 17

[Martin Schröder [:mschroeder]]

Comment on attachment 342793 [details] [diff] [review]
Step 2 - Move entities and rules around

r=mschroeder

Comment 18

[Philipp Kewisch [:Fallen] [:📆][:🧩]]

Attachment: Step 1 - Move around files - v2

Debitrotted patch

Comment 19

[Philipp Kewisch [:Fallen] [:📆][:🧩]]

Attachment: Step 1 - No whitespaces

Comment 20

[Daniel Boelzle [:dbo]]

Comment on attachment 343902 [details] [diff] [review]
Step 1 - Move around files - v2

r=dbo

Comment 21

[Philipp Kewisch [:Fallen] [:📆][:🧩]]

Pushed to comm-central

Step 1: <http://hg.mozilla.org/comm-central/rev/afd7891715ee>
Step 2: <http://hg.mozilla.org/comm-central/rev/fdea86912f93>

I decided to check in the two side by side, I'll take care of breakages if they show up.

-> FIXED

Comment 22

[Philipp Kewisch [:Fallen] [:📆][:🧩]]

Attachment: [checked in] Regression fix - #1

I'm checking in this additional fix, without it the sunbird customize toolbar dialog is borked.

Comment 23

[Philipp Kewisch [:Fallen] [:📆][:🧩]]

Additional fix is changeset <http://hg.mozilla.org/comm-central/rev/5bf762b71c0d>

Comment 24

[Philipp Kewisch [:Fallen] [:📆][:🧩]]

These bugs are likely targeted at Lightning 1.0b1, not Lightning 1.0. If this change was done in error, please adjust the target milestone to its correct value. To filter on this bugspam, you can use "lightning-10-target-move".