[ Thread Index |
Date Index
| More lists.liballeg.org/allegro-developers Archives
]
On April 12, 2010, Thomas Fjellstrom wrote:
> On April 11, 2010, Matthew Leverton wrote:
> > On Mon, Apr 12, 2010 at 12:33 AM, Peter Wang <novalazy@xxxxxxxxxx> wrote:
> > > I think al_init_acodec_addon() should be required to maintain the
> > > separation.
> >
> > Currently it wouldn't do anything. That is, al_load_sample() already
> > automatically calls it.
> >
> > >> * "foo" autodetect full name => "libfoo.so" (UNIX) or "foo.dll" or
> > >> "libfoo.dll" (Windows)
> > >> * "foo.dll" only try foo.dll, since the extension is present.
> > >
> > > Firstly, on Linux that's going to be a problem due to distribution
> > > quirks. On one distro, foo might be named libfoo.so, another might
> > > have it as libfoo-1.so. libfoo.so might be ABI-incompatible with the
> > > version that acodec was compiled with, e.g. different structure
> > > sizes.
> >
> > Distros would be free to specify a fully qualified name.
> >
> > cmake .. -DLIBFOO=libfoo-1.so
> >
> > > On Windows, I don't know if the problem is acute but I've seen
> > > different copies of FLAC around, under the names so similar problems
> > > might apply.
> >
> > Then the user already is experiencing "DLL Hell," and Allegro isn't
> > contributing to it. We can be careful to specify names that are
> > version qualified, if such a standard library exists. (e.g.,
> > png12.dll) We only are generally working with a known subset of
> > external libraries; I don't particularly care if the above
> > al_open_library functions, etc, are even part of the documented API.
> >
> > > DUMB's build system doesn't even build a shared library as far as I
> > > remember.
> >
> > I've updated it to support that.
> >
> > > I think the more robust solution is for acodec to load our own DLLs
> > > (like allegro_acodec-FLAC.dll) which link to FLAC. The number of DLLs
> > > will not increase if you statically link the dependencies into the
> > > Allegro modules.
> >
> > How does that help us know that flac.dll is the same across different
> > PCs?
>
> flac would be static linked into allegro_acodec-FLAC.dll in most cases.
> instant surety that it would work fine. In case that dynamic linking the
> dependency into the loadable modules is supported, dlopen/LoadLibrary
> will fail, and the format support won't be added. *
* if the flac dll isn't compatible.
>
> > --
> > Matthew Leverton
--
Thomas Fjellstrom
tfjellstrom@xxxxxxxxxx