Closed Bug 955395 Opened 10 years ago Closed 10 years ago

Directory change in Makefile to locate app-license.html causes file path problems.

Categories

(Instantbird Graveyard :: Other, defect)

All
Windows XP
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: wnayes, Assigned: wnayes)

References

Details

Attachments

(1 file, 1 obsolete file)

*** Original post on bio 1957 at 2013-05-15 17:27:00 UTC ***

Compiling Instantbird with pymake with Mozilla 20/21 causes a 'FILE_NOT_FOUND' error when the /instantbird/app/app-license.html file is processed. This is due to how the /instantbird/content/Makefile.in changes directories, as a call to 'pwd' returns a filepath different than what is expected (tested on Windows).
*** Original post on bio 1957 as attmnt 2429 at 2013-05-15 17:51:00 UTC ***

This moves the app-license.html to content and adjusts the Makefile.in to work with directories similar to how comm-central does, as discussed on IRC.
Attachment #8354196 - Flags: review?(florian)
*** Original post on bio 1957 as attmnt 2430 at 2013-05-15 21:07:00 UTC ***

This patch moves the app-license.html to content/overrides, which matches more closely the Thunderbird directories (in particular, mail/base/content/overrides).
Attachment #8354197 - Flags: review?(florian)
Comment on attachment 8354196 [details] [diff] [review]
Moving the app-license.html to content.

*** Original change on bio 1957 attmnt 2429 at 2013-05-15 21:07:41 UTC was without comment, so any subsequent comment numbers will be shifted ***
Attachment #8354196 - Attachment is obsolete: true
Attachment #8354196 - Flags: review?(florian)
Comment on attachment 8354197 [details] [diff] [review]
Moving the app-license.html to content/overrides

*** Original change on bio 1957 attmnt 2430 at 2013-05-15 21:25:23 UTC ***

Looks good by code inspection. I would like to check that this works correctly on non-Windows before pushing the patch.

I'm also not sure if this is something we need for 1.4 or if we should wait for after the release to push it.
Attachment #8354197 - Flags: review?(florian) → review+
Blocks: 955394
*** Original post on bio 1957 at 2013-05-23 02:35:58 UTC ***

I guess I'm unsure whether this is checkin-needed or not...but I think without that it'll fall through the cracks.
Whiteboard: [checkin-needed]
*** Original post on bio 1957 by Emanuel Hoogeveen <emanuel.hoogeveen AT gmail.com> at 2013-05-23 03:45:24 UTC ***

I may be missing something, but I don't think the attached patch actually moves app-license.html to its new location. I still got the error after applying attachment 8354197 [details] [diff] [review] (bio-attmnt 2430) locally until I moved it myself.
*** Original post on bio 1957 at 2013-05-23 03:47:19 UTC ***

(In reply to comment #5)
> I may be missing something, but I don't think the attached patch actually moves
> app-license.html to its new location. I still got the error after applying
> attachment 8354197 [details] [diff] [review] (bio-attmnt 2430) [details] locally until I moved it myself.

Sure it does:
> diff --git a/instantbird/app/app-license.html b/instantbird/content/overrides/app-license.html
> rename from instantbird/app/app-license.html
> rename to instantbird/content/overrides/app-license.html

If you apply the patch with patch instead of hg import (or are using git or something) this might not work.
*** Original post on bio 1957 by Emanuel Hoogeveen <emanuel.hoogeveen AT gmail.com> at 2013-05-23 04:01:31 UTC ***

Oh, I see - that part doesn't show up in the diff view for some reason. And then I missed it when glancing at the details view because there's no space between it and the makefile changes. Carry on :D
*** Original post on bio 1957 at 2013-05-23 23:13:22 UTC ***

Thanks Will!

http://hg.instantbird.org/instantbird/rev/97a56cf95247
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Whiteboard: [checkin-needed]
Target Milestone: --- → 1.5
Blocks: 955433
No longer blocks: 955433
You need to log in before you can comment on or make changes to this bug.