Re: [Simulavr-devel] build patch

[reinhard . jessich]

On Sat, 05 Jan 2002, Theodore A. Roth wrote:
> 
> I've taken Reinhard's build fixes and merged them. I did change a few 
> things though. I haven't check this in yet in case anyone finds problems 
> with it.
I got the patch with CR,LF line endings? Have you sent it like this?
I did a dos2unix befor I patched the SW.

In my last patch I have forgotten to add the "-VERSION" to the help text of the
--with-docdir configure option.

> - The dist-hook rule in top level Makefile.am isn't needed. Plus, the way
> I wrote it, it would copy the files to the wrong place (build/simulavr/)
> and leave some garbage behind. The file in $auxdir are still ending up in
> the tarball.
> 
> - Added a check for avr-ld in avr_binutils.m4.
> 
> - test_asm/test_8515/Makefile.am: I did this a little different than 
> Reinhards patch. The $(TEST_ASM_SRC:.asm=.bin) construct is a gnumake only 
> feature (as far as I know), so they should all be prefixed by the 
> @ifGNUmake@ macro. I also went ahead and added the code to perform the 
> final linking which I talked about last night. Since this affects the 
> build, I thought it appropriate to add it now so Reinhard can test it.
I had problems with the new avr-ld command it exits with the following error:
  test_blink.asm:15: undefined reference to `MAIN'

Why you haven't discoverd this error with your avr_ld?

I have a MAIN line in the elf file. Here is the output of avr-objdump
  00000000         *UND*  00000000 MAIN
and also a relocation
  00000000 R_AVR_13_PCREL    MAIN

So avr-ld complains correctly.

Then I have changed all labels (two) in test_blink.asm to capital letters,
because in all other asm files they are in capital letters.

Now I could make all without any error.

Attached is a patch for this changes. You have to apply it after the
sim-12-13-pre2-build.patch.

> I did the following and everything looks good:
> 
>   $ cd build-sim
>   $ ../simulavr/configure --prefix=$HOME/local/sim
>   $ make
There must be a differece in your avr tools and mine. Or you haven't added the
changes in test_blink.asm to the patch or ... .

>   $ make install
>   $ make uninstall
>   $ make dist
>   $ make distcheck
> 
> =====================================================
> simulavr-0.0.13-pre3.tar.gz is ready for distribution
> =====================================================
With my patch applied I got it also.

> 
> Reinhard,
> 
> If you don't have any problems with this, I'll make up a complete 13-pre3 
> patch with everything from version 0.0.12 to present.
> 
> I'd like to get version 13 out by tuesday or wednesday. I have to make a 
> trip to southern California for about a week. I'm planning to leave 
> thursday morning. If you guys have anything you want in 13, I'll need to 
> have the patch by tuesday to ensure it gets in.
I hope the gdb patch is also ready until this date. If so, you could add it to
the 0.0.13 package.

> Simulavr is really starting to solidify. Once the gdb interface and build 
> systems are rock solid, I'll have to bump it up to 0.1.0. I'm thinking 
> that the 0.1 goal should be working on adding the missing simulation 
> features.
Yes, we should start with this!
I have already started to think about the external interface requirements.

Reinhard

-- 
 Ing. Reinhard Jessich              mailto: address@hidden
 A-1190 Vienna, Goergengasse 2/2/1  phone: +43/1/3692600
 http://members.telering.at/jessich mobile: +43/664/1735439

sim-12-13-pre2-build-jess.patch
Description: Text document