Opened 18 years ago
Closed 18 years ago
#2042 closed task (fixed)
Links-2.1pre23
| Reported by: | Randy McMurchy | Owned by: | |
|---|---|---|---|
| Priority: | normal | Milestone: | 6.2.0 |
| Component: | BOOK | Version: | SVN |
| Severity: | normal | Keywords: | |
| Cc: |
Description
Version increment to 2.1pre22
Change History (7)
comment:1 by , 18 years ago
comment:2 by , 18 years ago
| Summary: | Links-2.1pre22 → Links-2.1pre23 |
|---|
comment:3 by , 18 years ago
| Owner: | changed from to |
|---|---|
| Status: | new → assigned |
comment:4 by , 18 years ago
Does anyone know why we're suppressing warnings for this package? It's nice, but we don't do it anywhere else in the book. And the -O2 overrides the default CFLAGS used in the package, something we don't do anywhere else either.
comment:5 by , 18 years ago
I did the warning suppression. configure's default CFLAGS in the package in the book is CFLAGS="-g -O2" I was just doing a touchup for sloppy programming that was bugging me. I thought there was some educational content there too.
I have done this for at least one other package. See the sed in OpenSSL. Its not a requirement for an editor to do this type of thing, but I think its a nice touch when the fix is relatively obvious and easy.
comment:6 by , 18 years ago
I built the package after I wrote that bit above. Wow, are there a lot of compiler warnings. It's hard to even find the actual commands in the output. I'll leave it alone, nothing bad can come of it.
Current version is 2.1pre23.