postflight on macosx fails when building thunderbird with preinstalled lightning

VERIFIED FIXED in 1.0b1

Status

defect
VERIFIED FIXED
11 years ago
10 years ago

People

(Reporter: ause, Assigned: ause)

Tracking

unspecified
1.0b1
All
macOS
Bug Flags:
tb-integration +

Details

Attachments

(1 attachment, 1 obsolete attachment)

the unify step breaks because the install.rdf of lightning contain different platform entries.
Flags: tb-integration?
seems like something we want, since there's a patch - you should request review from a module owner or peer for the module this file is in.
Flags: tb-integration? → tb-integration+
Comment on attachment 343385 [details] [diff] [review]
merge lightning install.rdf before running unify

i will come up with another patch as this one will break builds without --enable-calendar
Attachment #343385 - Attachment is obsolete: true
requires moving merge-installrdf.py in the same commit
Attachment #343537 - Flags: review?
Attachment #343537 - Flags: review? → review?(gozer)
(In reply to comment #4)
> Created an attachment (id=343537) [details]
> the new version also includes the suggested move of merge-installrdf.py from
> calendar/lightning/build to build
> 
> requires moving merge-installrdf.py in the same commit

Given you're in hg (and with git patches turned on), you can do hg rename yourself to move the file, and then it will show up in hg diff (or hg qdiff if you're using mq as well), you can attach a patch, and if it isn't you pushing it, then as long as the person uses hg import or hg qimport to get it into their repository it will work fine.
Attachment #343537 - Flags: review?(gozer) → review?(gozer)
Assignee: nobody → ause
Status: NEW → ASSIGNED
Is there any way this logic could somehow be kept inside calendar/lightninig/build/*.mk instead of inserting itself into build/* ?

Or possibly better/different, generalize the approach to correctly unify all install.rdf's generated during universal builds (not being lightninig specific), as this is a generic problem with building universal extensions with arch-specific bits.
(In reply to comment #6)
> Is there any way this logic could somehow be kept inside
> calendar/lightninig/build/*.mk instead of inserting itself into build/* ?
i don't think that makes sense as the inserted actions are required for tb packing, not for creating a lightning.xpi.

> 
> Or possibly better/different, generalize the approach to correctly unify all
> install.rdf's generated during universal builds (not being lightninig
> specific), as this is a generic problem with building universal extensions with
> arch-specific bits.
sure, the applied patch is just a workaround for the current problem until unify learns how to merge install.rdf. if you like i can try to create for unify. but as this is hosted in another repository and i doubt that we want to duplicate it, this seemed to be the most promising way to get integrated builds for the moment.
(In reply to comment #7)
> (In reply to comment #6)
> > Is there any way this logic could somehow be kept inside
> > calendar/lightninig/build/*.mk instead of inserting itself into build/* ?
> i don't think that makes sense as the inserted actions are required for tb
> packing, not for creating a lightning.xpi.

Yes, but what I was trying to say is that it's definitely a workaround for lightning. And under universal builds, any module like lightning would end up requiring this exact same workaround.

> > 
> > Or possibly better/different, generalize the approach to correctly unify all
> > install.rdf's generated during universal builds (not being lightninig
> > specific), as this is a generic problem with building universal extensions with
> > arch-specific bits.
> sure, the applied patch is just a workaround for the current problem until
> unify learns how to merge install.rdf.

Okay, if this was an intended workaround from the start, and teaching unify how to merge install.rdf's, then I am okay with it. r=gozer
Attachment #343537 - Flags: review?(gozer) → review+
Attachment #343537 - Attachment description: the new version also includes the suggested move of merge-installrdf.py from calendar/lightning/build to build → [checked in] the new version also includes the suggested move of merge-installrdf.py from calendar/lightning/build to build
Blocks: 401779
i submitted Bug 461515 to enhance unify, not sure who to bother with that.
No longer blocks: 401779
ause: Can this bug be marked FIXED?
afaik, preinstalled lightning is still disabled by build configuration. so i'd like to wait until it's active.
DISABLE_LIGHTNING_INSTALL is gone now, lightning boxes build thunderbird with preinstalled lightning without error
Status: ASSIGNED → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Status: RESOLVED → VERIFIED
Target Milestone: 1.0 → 1.0b1
You need to log in before you can comment on or make changes to this bug.