Discussion:
[9fans] Loader dependencies
(too old to reply)
Steven Stallion
2013-04-10 05:04:11 UTC
Permalink
gunge% mk
mk: don't know how to make '../8l/elf.h' in directory
/usr/stallion/src/armv7a/5l

... really? I assume this crept in as a part of Geoff's latest ELF changes.
David du Colombier
2013-04-10 05:49:07 UTC
Permalink
Post by Steven Stallion
gunge% mk
mk: don't know how to make '../8l/elf.h' in directory
/usr/stallion/src/armv7a/5l
mkdir /usr/stallion/src/armv7a/8l
cp /sys/src/cmd/8l/elf.? /usr/stallion/src/armv7a/8l

elf.c and elf.h are platform-independent files and
are currently located in the 8l directory because
it hasn't seemed worth creating /sys/src/cmd/ld
just for two files.
--
David du Colombier
l***@proxima.alt.za
2013-04-10 05:58:38 UTC
Permalink
Post by David du Colombier
elf.c and elf.h are platform-independent files and
are currently located in the 8l directory because
it hasn't seemed worth creating /sys/src/cmd/ld
just for two files.
libelf.a?

++L
Steven Stallion
2013-04-10 06:05:22 UTC
Permalink
Nah, ld is warranted. I've posted a patch somewhat recently to add uImage
support that would fit in as well. Geoff had mentioned an interest in
adding support to each of the loaders rather than just 5l.
Post by l***@proxima.alt.za
Post by David du Colombier
elf.c and elf.h are platform-independent files and
are currently located in the 8l directory because
it hasn't seemed worth creating /sys/src/cmd/ld
just for two files.
libelf.a?
++L
Charles Forsyth
2013-04-10 09:19:30 UTC
Permalink
That wouldn't, however, have solved your immediate problem: you'd have got
a missing ld instead of missing 8l
Post by Steven Stallion
Nah, ld is warranted. I've posted a patch somewhat recently to add uImage
support that would fit in as well. Geoff had mentioned an interest in
adding support to each of the loaders rather than just 5l.
Post by l***@proxima.alt.za
Post by David du Colombier
elf.c and elf.h are platform-independent files and
are currently located in the 8l directory because
it hasn't seemed worth creating /sys/src/cmd/ld
just for two files.
libelf.a?
++L
Steven Stallion
2013-04-10 05:59:50 UTC
Permalink
I understand the taxonomy. I'm commenting on the general laziness of the
change. Given that many of us tend to work on acl changes in isolation,
this is an annoyance at best.
Post by David du Colombier
Post by Steven Stallion
gunge% mk
mk: don't know how to make '../8l/elf.h' in directory
/usr/stallion/src/armv7a/5l
mkdir /usr/stallion/src/armv7a/8l
cp /sys/src/cmd/8l/elf.? /usr/stallion/src/armv7a/8l
elf.c and elf.h are platform-independent files and
are currently located in the 8l directory because
it hasn't seemed worth creating /sys/src/cmd/ld
just for two files.
--
David du Colombier
erik quanstrom
2013-04-10 14:01:39 UTC
Permalink
Post by Steven Stallion
I understand the taxonomy. I'm commenting on the general laziness of the
change. Given that many of us tend to work on acl changes in isolation,
this is an annoyance at best.
i think you misspelled "conservitivism." ☺ imo, the elf patch
did a complete job of consolidating elf support, and
in some cases extending it to all elf-supporing arches.

what it didn't do is create an ld directory, but that's
a seperate issue. and i could understand resisting adding it.

but i agree that an ld directory is probablly justified now,
and would simplify maintence. for example, the recent
library search change that touched every linker, not just
the ld directory. (there are more than 2 files that would
be in it.) maybe it would also help for uimage support.

i considered doing this a few weeks ago, but the potential
differences with sources kept me from it.

regardless of the directory name, in your case the fix is
to copy the donor directory to the same parent as your
test linker.

- erik

Loading...