Author Topic: Error compiling for 40D  (Read 3129 times)

jplxpto

  • Developer
  • Hero Member
  • *****
  • Posts: 506
Error compiling for 40D
« on: October 04, 2012, 12:25:23 AM »


../../src/bmp.c: In function 'bmp_draw_scaled_ex':
../../src/bmp.c:1220:27: warning: pointer targets in initialization differ in signedness [-Wpointer-sign]
../../src/bmp.c:1276:27: warning: pointer targets in initialization differ in signedness [-Wpointer-sign]
[ CC       ]   font-dyn.o
[ CC       ]   config.o
[ CC       ]   menu.o
../../src/menu.c: In function 'handle_ml_menu_keys':
../../src/menu.c:2204:5: error: duplicate case value
../../src/menu.c:2187:5: error: previously used here
make: *** [menu.o] Error 1


parent: 4885:073d5b5731eb

jplxpto

  • Developer
  • Hero Member
  • *****
  • Posts: 506
Re: Error compiling for 40D
« Reply #1 on: October 04, 2012, 12:31:46 AM »
parent: 4885:073d5b5731eb


../../src/bmp.c: In function 'bmp_draw_scaled_ex':
../../src/bmp.c:1220:27: warning: pointer targets in initialization differ in signedness [-Wpointer-sign]
../../src/bmp.c:1276:27: warning: pointer targets in initialization differ in signedness [-Wpointer-sign]
[ CC       ]   font-dyn.o
[ CC       ]   config.o
[ CC       ]   menu.o
../../src/menu.c: In function 'handle_ml_menu_keys':
../../src/menu.c:2204:5: error: duplicate case value
../../src/menu.c:2187:5: error: previously used here
make: *** [menu.o] Error 1


...

bmp.o: In function `set_ml_palette_if_dirty':
bmp.c:(.text+0x88c): undefined reference to `BmpDDev_take_semaphore'
bmp.c:(.text+0x898): undefined reference to `BmpDDev_give_semaphore'
bmp.c:(.text+0x8a4): undefined reference to `PB_Palette'
collect2: ld returned 1 exit status



40D v1.1.1

Quote

NSTUB(0xFFCB9078, BmpDDev_give_semaphore);
NSTUB(0xFFCB9028, BmpDDev_take_semaphore);

NSTUB(0x1E388, PB_Palette)


nanomad

  • Administrator
  • Hero Member
  • *****
  • Posts: 2918
  • All your websites are belong to us
Re: Error compiling for 40D
« Reply #2 on: October 04, 2012, 12:43:20 AM »
Little trick to find out the offending revision:)

Code: [Select]
hg bisect --reset
hg bisect --bad [revision with error]
hg bisect --good [revision that works fine]
hg bisect --command 'make clean && make' <- in 40D directory
EOS 1100D | EOS 650 (No, I didn't forget the D) | Ye Olde Canon EF Lenses ('87): 50 f/1.8 - 28 f/2.8 - 70-210 f/4 | EF-S 18-55 f/3.5-5.6 | Metz 36 AF-5

nanomad

  • Administrator
  • Hero Member
  • *****
  • Posts: 2918
  • All your websites are belong to us
Re: Error compiling for 40D
« Reply #3 on: October 04, 2012, 12:50:01 AM »
I've pushed a temporary fix for the duplicate button code (you're defining BGMT_Q as BGMT_PRESS_DIRECT_PRINT, i guess beacuse the 40D doesn't have a Q button). Now you need to find  three stubs:
Code: [Select]
bmp.c:(.text+0x920): undefined reference to `BmpDDev_take_semaphore'
bmp.c:(.text+0xa34): undefined reference to `BmpDDev_give_semaphore'
bmp.c:(.text+0xa40): undefined reference to `PB_Palette'

Code: [Select]
Changeset 4855:33e5e0770ed7: bad
The first bad revision is:
changeset:   4855:33e5e0770ed7
branch:      unified
user:        alex@thinkpad
date:        Tue Oct 02 12:28:52 2012 +0300
summary:     5Dc: enabled a custom color palette for ML
EOS 1100D | EOS 650 (No, I didn't forget the D) | Ye Olde Canon EF Lenses ('87): 50 f/1.8 - 28 f/2.8 - 70-210 f/4 | EF-S 18-55 f/3.5-5.6 | Metz 36 AF-5

jplxpto

  • Developer
  • Hero Member
  • *****
  • Posts: 506
Re: Error compiling for 40D
« Reply #4 on: October 04, 2012, 01:14:59 AM »
I've pushed a temporary fix for the duplicate button code (you're defining BGMT_Q as BGMT_PRESS_DIRECT_PRINT, i guess beacuse the 40D doesn't have a Q button). Now you need to find  three stubs:
Code: [Select]
bmp.c:(.text+0x920): undefined reference to `BmpDDev_take_semaphore'
bmp.c:(.text+0xa34): undefined reference to `BmpDDev_give_semaphore'
bmp.c:(.text+0xa40): undefined reference to `PB_Palette'

Code: [Select]
Changeset 4855:33e5e0770ed7: bad
The first bad revision is:
changeset:   4855:33e5e0770ed7
branch:      unified
user:        alex@thinkpad
date:        Tue Oct 02 12:28:52 2012 +0300
summary:     5Dc: enabled a custom color palette for ML


Thank you Nanomad! I found the missing stubs ... and thanks for the tip ... the hg is a new tool for me :)
Can you help code ?!... Please...

Yes, The camera 40D doesn't have a Q button