237799 - Calendar under OS/2

Status: RESOLVED FIXED

(Calendar :: General, defect)

Description

[Andy Willis (abwillis)]

User-Agent:       Mozilla/5.0 (OS/2; U; Warp 4.5; en-US; rv:1.7b) Gecko/20040314
Build Identifier: Mozilla/5.0 (OS/2; U; Warp 4.5; en-US; rv:1.7b) Gecko/20040314

This is a bug to submit my patches to get Calendar working under OS/2.

Reproducible: Always
Steps to Reproduce:
I have now gotten calendar to work under OS/2.

Comment 1

[Andy Willis (abwillis)]

Attachment: allmakefiles.sh

allmakefiles has calendar being built before the other-licenses which on OS/2
causes an error that mozillas_s.a cannot be found.  I just reversed them but as
I don't know how it was working this way on the other systems I don't know that
reversing them won't break other platforms.  I don't know shell scripting to
write proper ifdef's in it.

Comment 2

[Andy Willis (abwillis)]

Attachment: createxpi.sh

Comment 3

[Andy Willis (abwillis)]

Attachment: makefile.in from calendar diretory

The objdir could use some massaging.

Comment 4

[Andy Willis (abwillis)]

Attachment: OS/2 xpi creator.

I use this file to create the xpi on OS/2 as make xpi didn't do anything.  On
OS/2 I am not getting a makefile created in the Calendar directory, I am having
to assume at this point that this is due to the way that configure works as I
don't really understand it.

Comment 5

[Andy Willis (abwillis)]

Attachment: calender.ico

This needs to be put into mozilla/calendar/os2/icons/default that needs to be
built created as well.

Comment 6

[Andy Willis (abwillis)]

Attachment: makefile.in in libxpical directory

I don't understand really why this was needed.	On OS/2 we get mozicals.lib
instead of mozicalss that I assume it has something to do with OS/2 having a
8.3 dll restriction.

Comment 7

[Andy Willis (abwillis)]

Attachment: makefile.in from caledar directory

I figured out the problem with the makefile not being created, it was my
mistake here so I corrected the makefile.in
Andy

Comment 8

[Andy Willis (abwillis)]

Attachment: calendar makefile.in

Realised that if platform was windows that it would be set then it would be
changed to linux.  A more elegant method would be better than this.

Comment 9

[Andy Willis (abwillis)]

Attachment: makefile.in in calendar directory.

One more change to makefile.in, OS/2 needs to specify to run perl for .pl
files.

Comment 10

[Mostafa Hosseini]

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

Comment 11

[Mike Kaply [:mkaply]]

Nice.

That $(PERL) change in the last diff can probably be done for everyone.

I'll these changes in tomorrow if I can.

Comment 12

[jhp (no longer active)]

Andy, the allmakefiles.sh change should be unnecessary.  The order in that file
does not affect compilation order.  This file only exists to create the
"Makefile" from the "Makefile.in".  The compilation order of those directories
is handled by the top level Makefile.in.  And in fact, that Makefile.in has this
line:

   ifdef MOZ_CALENDAR
   tier_98_dirs    += other-licenses/libical calendar
   endif

So it should already be building 'other-licenses/libical' before 'calendar'.

Comment 13

[Andy Willis (abwillis)]

(In reply to comment #12)

> 
> So it should already be building 'other-licenses/libical' before 'calendar'.

I will change it back and verify.  When I first got the error I did a search on
google and found a reference that said to look for *mozical* and I didn't have
one.  I changed the order and got one but it then failed due to the extra (or
missing depending on perspective) s.  I didn't change the order back after
changing everything else.
Andy

Comment 14

[Andy Willis (abwillis)]

(In reply to comment #12)
> Andy, the allmakefiles.sh change should be unnecessary. 
I have verified that it indeed does work with the original allmakefiles.sh,
which would explain why I couldn't understand it working on the others and not
OS/2 :)
Andy

Comment 15

[Andy Willis (abwillis)]

Comment on attachment 144143 [details]
OS/2 xpi creator.

This is not necessary, I used it to create the xpi before I got make xpi
working.

Comment 16

[Mike Kaply [:mkaply]]

Nice work Andy.

Everything should be in with some subtle changes.

FYI,

1. that $(PERL) should have been used for everyone - no need to if it (look
earlier in the file)

2. the fact that SHORT_LIBNAME was definied for the libicalss was incorrect. We
only use SHORT_LIBNAME for DLLs and libicalss is a static lib. So instead of
your change, I just removed the SHORT_LIBNAME code.

I checked in the icon.

Thanks for your help motivating me to do this work. It's been on our list forever.

I'm marking this bug fixed.

Comment 17

[Reed Loden [:reed]]

The bugspam monkeys have been set free and are feeding on Calendar :: General. Be afraid for your sanity!