sox-devel@lists.sourceforge.net unofficial mirror
 help / color / mirror / code / Atom feed
* [PATCH] Makefile.am: export internal lsx_error
@ 2018-02-23 22:42 Ioan-Adrian Ratiu
  2018-02-25  3:12 ` Eric Wong
  0 siblings, 1 reply; 2+ messages in thread
From: Ioan-Adrian Ratiu @ 2018-02-23 22:42 UTC (permalink / raw)
  To: sox-devel

commit 5c58413544 ("Don't export (most) internal libsox symbols")
breaks dynamic flac builds as flac.c references lsx.error, so add it
to the list of exceptions.

| .libs/flac.o: In function `decoder_read_callback':
| /usr/src/debug/sox/14.4.2-r0/sox-14.4.2/src/flac.c:63: undefined reference to `lsx_error'

Signed-off-by: Ioan-Adrian Ratiu <adi@adirat.com>
---
 src/Makefile.am | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/Makefile.am b/src/Makefile.am
index 7cceaafd..64ac188e 100644
--- a/src/Makefile.am
+++ b/src/Makefile.am
@@ -95,7 +95,7 @@ libsox_la_LIBADD += @GOMP_LIBS@
 
 libsox_la_CFLAGS = @WARN_CFLAGS@
 libsox_la_LDFLAGS = @APP_LDFLAGS@ -version-info @SHLIB_VERSION@ \
-  -export-symbols-regex '^(sox_.*|lsx_(check_read_params|(close|open)_dllibrary|(debug(_more|_most)?|fail|report|warn)_impl|eof|fail_errno|filelength|find_(enum_(text|value)|file_extension)|getopt(_init)?|lpc10_(create_(de|en)coder_state|(de|en)code)|raw(read|write)|read(_b_buf|buf|chars)|realloc|rewind|seeki|sigfigs3p?|strcasecmp|tell|unreadb|write(b|_b_buf|buf|s)))$$'
+  -export-symbols-regex '^(sox_.*|lsx_(check_read_params|(close|open)_dllibrary|(debug(_more|_most)?|fail|report|warn)_impl|eof|error|fail_errno|filelength|find_(enum_(text|value)|file_extension)|getopt(_init)?|lpc10_(create_(de|en)coder_state|(de|en)code)|raw(read|write)|read(_b_buf|buf|chars)|realloc|rewind|seeki|sigfigs3p?|strcasecmp|tell|unreadb|write(b|_b_buf|buf|s)))$$'
 
 if HAVE_WIN32_LTDL
   libsox_la_SOURCES += win32-ltdl.c win32-ltdl.h
-- 
2.16.2


------------------------------------------------------------------------------
Check out the vibrant tech community on one of the world's most
engaging tech sites, Slashdot.org! http://sdm.link/slashdot
_______________________________________________
SoX-devel mailing list
SoX-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/sox-devel

^ permalink raw reply related	[flat|nested] 2+ messages in thread

* Re: [PATCH] Makefile.am: export internal lsx_error
  2018-02-23 22:42 [PATCH] Makefile.am: export internal lsx_error Ioan-Adrian Ratiu
@ 2018-02-25  3:12 ` Eric Wong
  0 siblings, 0 replies; 2+ messages in thread
From: Eric Wong @ 2018-02-25  3:12 UTC (permalink / raw)
  To: sox-devel

Ioan-Adrian Ratiu <adi@adirat.com> wrote:
> commit 5c58413544 ("Don't export (most) internal libsox symbols")
> breaks dynamic flac builds as flac.c references lsx.error, so add it
> to the list of exceptions.

Thanks, but Mans already added that exception in
commit 600c291ab00f4afb2941cd93f69942fe395f3e8a
("Update exported symbol list")

https://bogomips.org/sox.git ("pu" branch) and
https://github.com/mansr/sox.git have it.
(neither of us have permissions on Sourceforge git)

------------------------------------------------------------------------------
Check out the vibrant tech community on one of the world's most
engaging tech sites, Slashdot.org! http://sdm.link/slashdot
_______________________________________________
SoX-devel mailing list
SoX-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/sox-devel

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2018-02-25  3:13 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2018-02-23 22:42 [PATCH] Makefile.am: export internal lsx_error Ioan-Adrian Ratiu
2018-02-25  3:12 ` Eric Wong

Code repositories for project(s) associated with this public inbox

	https://80x24.org/mirrors/sox.git

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).