View Issue Details
| ID | Project | Category | View Status | Date Submitted | Last Update |
|---|---|---|---|---|---|
| 0001930 | unreal | installing | public | 2004-07-08 05:54 | 2004-07-18 18:24 |
| Reporter | fez | Assigned To | |||
| Priority | normal | Severity | feature | Reproducibility | always |
| Status | resolved | Resolution | fixed | ||
| Platform | x86 | OS | Linux | OS Version | 2.4.x |
| Product Version | 3.2.1 | ||||
| Summary | 0001930: a suggestion for Config and src/modules/Makefile.in | ||||
| Description | well, i've given this some thought... I requested a while back that a Config option be added to compile only commands.so instead of that and m_*.so, and it was considered but decided against because it was thought it wouldn't bring that much of an improvement,,,, Well I got to thinking, the main reason i made that request is because of the fact that the src/modules/Makefile[.in] makes a single call to gcc to compile all of commands.so, without compiling each .c to a .o first, and as a result it takes FOREVER (and on my P166, which gets kind of touchy and sometimes has the nasty sig11 problem, it sometimes takes several attempts at make to get it compiled and it takes FOREVERPLUSONE).... soo..... I've brought my request down a notch - instead of requesting that only commands.so be compiled, i'd just request that we at least compile each .c into a .o before linking. This has a few benefits: 1. that single call to gcc for commands.so doesn't take FOREVER 2. if a user/module coder wanted to make just a small change to one of the commands.so commands like m_whois.c, they wouldn't have to recompile ALL the commands to make the change take effect. make would see that only m_whois.o needs to be updated, then just relink commands.so from the already existing .o's 3. i think it's good practice to have all the .o's anyway, and it would also potentially make it easier to in the future implement a ./Config option to only compile commands.so (although -DDYNAMIC_LINKING isn't used when compiling the .o's, only the .so's..). I've attached the proposed src/modules/Makefile.in for you to review, with any changes duly noted Thoughts? - fez | ||||
| Tags | No tags attached. | ||||
| Attached Files | |||||
| 3rd party modules | |||||
|
|
Yeah, doesn't sound too bad. (although the if-the-user-modified-unreal isn't exactly a good argument to me since I just beat those guys to death ;p.. but it would be useful to myself & lowmem boxes) |
|
|
Added in .102 |
| Date Modified | Username | Field | Change |
|---|---|---|---|
| 2004-07-08 05:54 | fez | New Issue | |
| 2004-07-08 05:54 | fez | File Added: Makefile.in | |
| 2004-07-08 11:13 | syzop | Note Added: 0006926 | |
| 2004-07-12 15:47 |
|
Status | new => assigned |
| 2004-07-12 15:47 |
|
Assigned To | => codemastr |
| 2004-07-18 18:24 |
|
Status | assigned => resolved |
| 2004-07-18 18:24 |
|
Resolution | open => fixed |
| 2004-07-18 18:24 |
|
Note Added: 0007142 |
