First Last Prev Next    No search results available      Search page      Enter new bug
Bug#: 219896
Alias:
Product:
Component:
Status: RESOLVED
Resolution: FIXED
Assigned To: Jack Lloyd <lloyd@randombit.net>
Hardware:
OS:
Version:
Priority:
Severity:
Reporter: Nickolay Kolchin-Semyonov <nbkolchin@gmail.com>
Add CC:
CC:
Remove selected CCs
URL:
Summary:
Status Whiteboard:
Keywords:

Filename Description Type Creator Created Size Actions
botan-ull.patch add ULL postfix to (hopefully) all 64bit constants in Botan. patch Nickolay Kolchin-Semyonov 2008-05-01 07:02 0000 201.68 KB Details | Diff
botan_gcc42_fix.patch Patch for configure.pl patch Jack Lloyd 2008-05-01 13:00 0000 815 bytes Details | Diff
Create a New Attachment (proposed patch, testcase, etc.) View All

Bug 219896 depends on: Show dependency tree
Show dependency graph
Bug 219896 blocks:
Votes: 0    Show votes for this bug    Vote for this bug

Additional Comments: (this is where you put emerge --info)







View Bug Activity   |   Format For Printing   |   XML   |   Clone This Bug


Description:   Opened: 2008-05-01 07:00 0000
BUG 117548 strikes back! 

error: integer constant is too large for ‘long’ type

P.S. Somebody, please fix misc/config/cc/gcc -mcpu settings. I don't completely
understand Gentoo policy on C-flags to do that.

Reproducible: Always

Steps to Reproduce:

------- Comment #1 From Nickolay Kolchin-Semyonov 2008-05-01 07:02:45 0000 -------
Created an attachment (id=151486) [edit]
add ULL postfix to (hopefully) all 64bit constants in Botan.

Patch was created against 1.7.5 version, but applies to 1.6.1.

------- Comment #2 From Jack Lloyd 2008-05-01 12:58:15 0000 -------
The problem is configure.pl will recognize that (say) GCC 4.1 needs
-fpermissive, but not GCC 4.2 (yes, this should be a compile-time test instead
of hardcoded, one of the many infelicities of the build system, it's just a
version test). Patch is commited to the stable branch
(net.randombit.botan.stable), I will attach to this PR as well.

------- Comment #3 From Jack Lloyd 2008-05-01 13:00:22 0000 -------
Created an attachment (id=151507) [edit]
Patch for configure.pl

------- Comment #4 From Alon Bar-Lev 2008-05-01 19:05:58 0000 -------
Should I apply only attachment#151507 [edit]? Nickolay, can you please confirm?

------- Comment #5 From Nickolay Kolchin-Semyonov 2008-05-01 19:40:17 0000 -------
(In reply to comment #4)
> Should I apply only attachment#151507 [edit] [edit]? Nickolay, can you please confirm?
> 

AFAIK, Lloyd is official Botan maintainer. So his patch is preferable.

------- Comment #6 From Alon Bar-Lev 2008-05-02 06:56:55 0000 -------
Added this patch.
But Jack, I have another issue with this package.
It looks like it chooses to use g++ and not ${CHOST}-g++ as expected. I tried
to analyze your configure.pl script but it is not standard (Why don't you use
autoconf/automake/)... Can you please fix this too?

------- Comment #7 From Alon Bar-Lev 2008-05-02 07:04:52 0000 -------
Made it work... In the future please separate the AR flags from the AR command.
But it would be great if you enable passing host prefix into configure like
autoconf.
Thanks!

------- Comment #8 From Jack Lloyd 2008-05-02 15:10:54 0000 -------
Autoconf/automake are avoided because while they solve some problems on Unix
they create many more for Windows users, and b/c autoconf's support for
compilers that aren't GCC is pitiful: what is currently ~1K lines of Perl would
have to become ~3K lines of configure.ac :( Sadly there are no good solutions
here (that I have found), believe me I don't like maintaining my own build
system any more than people like using it.

AR flags problem noted. You can override compiler binary name via CXX variable
(eg CXX=<CHOST>-g++ make); is more needed here?

------- Comment #9 From Alon Bar-Lev 2008-05-02 15:14:56 0000 -------
Thanks!
I hope you add the host prefix into configure... It will make it simpler...
Just mimic autoconf --host= --target= --build= parameters.
Also take the LIB_OPT from configure as well...

But for now we are OK.

First Last Prev Next    No search results available      Search page      Enter new bug