sox-devel@lists.sourceforge.net unofficial mirror
 help / color / mirror / code / Atom feed
From: Jan Stary <hans@stare.cz>
To: "Måns Rullgård" <mans@mansr.com>
Cc: sox-devel@lists.sourceforge.net
Subject: Re: Build system cleanup
Date: Fri, 21 Aug 2020 16:04:58 +0200	[thread overview]
Message-ID: <20200821140458.GA48586@www.stare.cz> (raw)
In-Reply-To: <yw1x5z9cp7vn.fsf@mansr.com>

> > (I would still like to get rid of auto* anyway :-)
> 
> Replacing it without losing anything is a huge amount of work.  The
> results can be great, but I don't want to spend the next month that
> right now.

Please have a look at https://github.com/janstary/sox
which uses a hand-written configure and does not depend on auto*.
The build procedure is described on that page (but boils down to
./confiugure; make ; make install).

It is far from complete, and quite behind all your recent fixes;
I will need to rebase that on the current master.

> > checking for gcc... gcc
> >
> > 	That would be $ gcc --version
> > 	gcc (GCC) 4.2.1 20070719
> 
> That's an OpenBSD problem, not a SoX problem.

The actual problem is that the build system looks for gcc,
instead of looking for cc, the C compiler
(which is clang be default here).

> > checking for mt... mt
> >
> > 	LOL, what?
> >
> > checking if mt is a manifest tool... no
> >
> > 	That's a shame.
> 
> It's something some systems require when linking.  That's all I know.

openbsd$ whatis mt
mt, eject(1) - magnetic tape and removable media manipulating program

debian$ man mt
NAME
       mt - control magnetic tape drive operation

> > checking for pkg-config... /usr/bin/pkg-config
> > checking pkg-config is at least version 0.9.0... yes
> >
> > 	Reading configure.ac, it's currently
> > 	oggvorbis, opus and flac that get detected via pkg-config;
> > 	for other formats, it checks for the header and the lib.
> > 	Is this distinction intentional?
> 
> Those libraries are sometimes installed in ways that _require_ unusual
> -I flags that we can't detect any other way (other than trying half a
> dozen variants until something works).  pkg-config provides the
> necessary information.  On the flip side, pkg-config can sometimes
> tell lies too, so I prefer to avoid it when possible.

Would you prefer, then, to detect flac, vorbis and opus
in the same way that the other external formats are detected,
i.e. searching for include.h and -llib?

> > checking for library containing pow... -lm
> > checking for library containing lrint... none required
> >
> > 	$ nm /usr/lib/libc.so.* | grep -Fw lrint 
> > 	$ nm /usr/lib/libm.so.* | grep -Fw lrint
> > 	00004a10 T lrint
> > 	00017980 T lrint
> >
> > 	So -lm _is_ required for lrint. Maybe it works by accident,
> > 	because -lm is pulled in for pow() anyway.
> >
> > checking for lrint... yes
> >
> > 	Yes, in libm.
> 
> On some systems it's all libc without a separate libm.

Yes, and ./configure is supposed to figure out
whether it is this or that case, right? On this system,
lrint is only present in -lm, which imho makes the detection

	checking for library containing lrint... none required

incorrect.

> Bear in mind that SoX predates many interfaces that are now standard.
> Much of the cruft was once necessary, or at least warranted.

Yes; I would like to get rid of it all.

> > 	$ pkg_info -L lame | grep .h$    
> > 	/usr/local/include/lame/lame.h
> >
> > 	OpenBSD installs third-party packages into /usr/local/,
> > 	but the configure script does not look there; it porbably
> > 	only searches in /usr, because that's where linux distributions
> > 	install their packages. (I'm not saying it should
> > 	-- I will test the same with -I added to the CPPFLAGS,
> > 	as the OpenBSD port of SoX does.)
> 
> If your system puts things outside the compiler's default search path,
> it is your responsibility to provide the information necessary to use
> them.  Setting CFLAGS or CPPFLAGS with the appropriate -I options is one
> way.  Depending on the compiler, setting C_INCLUDE_PATH might be another.

I am testing with CPPFLAGS and LDFLAGS, like the OpenBSD port of SoX does,
will report later.

> > checking for lpc10.h... no
> >
> > 	So why does it try to build lpc support later,
> > 	#including the lpc10.h that was not found?
> 
> I made a mistake.

Wolfgang reported it fixed but I don't see any other commit
on the branch; am I missign something obvious?

> > 	Ha, I didn't have to --disable-sunaudio as before.
> 
> Odd, I didn't intentionally change anything.

Some time ago, running just plain ./configure on OpenBSD
resulted in sunaudio being 'detected', because the check
was actually the presence of /usr/include/sys/audioio.h
(which does exist on OpenBSD); trying to actually build
sunaudio output device support then failed of course,
so I had to ./configure --without-sunaudio (now called
--disable-sunaudio). That's apparantly no longer needed,
the (non)presence of Sun Audio is correctly detected.

	At any rate, thank you very mich for the cleanup!

		Jan



_______________________________________________
SoX-devel mailing list
SoX-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/sox-devel

  reply	other threads:[~2020-08-21 14:05 UTC|newest]

Thread overview: 89+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2020-08-19 20:40 Build system cleanup Måns Rullgård
2020-08-20 20:40 ` Wolfgang Stoeggl via SoX-devel
2020-08-20 21:22   ` Måns Rullgård
2020-08-20 22:15     ` Wolfgang Stoeggl via SoX-devel
2020-08-21  8:24     ` Jan Stary
2020-08-21  8:16 ` Jan Stary
2020-08-21  8:31   ` Jan Stary
2020-08-21 10:28     ` Måns Rullgård
2020-08-21 12:24       ` Jan Stary
2020-08-21 12:46         ` Måns Rullgård
2020-08-21  8:49   ` Jan Stary
2020-08-21 10:33     ` Måns Rullgård
2020-08-21 12:49       ` Jan Stary
2020-08-21 12:50         ` Jan Stary
2020-08-21 13:32         ` Måns Rullgård
2020-08-21 14:09         ` Jan Stary
2020-08-21 11:15   ` Måns Rullgård
2020-08-21 14:04     ` Jan Stary [this message]
2020-08-21 14:47       ` Måns Rullgård
2020-08-21 19:26       ` Wolfgang Stoeggl
2020-08-21 20:20       ` Jan Stary
2020-08-21 15:16   ` Jan Stary
2020-08-21 15:21     ` Jan Stary
2020-08-21 17:08       ` Måns Rullgård
2020-08-21 17:43         ` Jan Stary
2020-08-21 17:52           ` Måns Rullgård
2020-08-21 17:57             ` Jan Stary
2020-08-21 20:17             ` Jan Stary
2020-08-21 20:45               ` Jan Stary
2020-08-21 21:43                 ` Måns Rullgård
2020-08-22  8:17                   ` Jan Stary
2020-08-22 10:10                     ` Måns Rullgård
2020-08-22 15:35                       ` Jan Stary
2020-08-22 15:44                         ` Jan Stary
2020-08-22 20:54                         ` Måns Rullgård
2020-08-23 14:07                           ` Jan Stary
2020-08-23 17:13                             ` Måns Rullgård
2020-08-23 19:46                               ` Jan Stary
2020-08-24 11:04                                 ` Måns Rullgård
2020-08-24 11:19                                   ` Jan Stary
2020-08-27  9:38                                     ` Jan Stary
2020-08-27 10:53                                       ` Måns Rullgård
2020-08-23 21:55                               ` Jan Stary
2020-08-24  6:56                                 ` Jan Stary
2020-08-27  9:40                                   ` Jan Stary
2020-08-27 11:08                                     ` Måns Rullgård
2020-08-27 11:37                                       ` Jan Stary
2020-08-27 11:53                                         ` Måns Rullgård
2020-08-27 13:05                                     ` Jan Stary
2020-08-27 13:31                                       ` Måns Rullgård
2020-08-27 14:10                                         ` Jan Stary
2020-08-27 14:38                                           ` Jan Stary
2020-08-27 15:11                                         ` Måns Rullgård
2020-08-27 17:41                                           ` Jan Stary
2020-08-27 18:50                                             ` Måns Rullgård
2020-08-27 19:18                                               ` Jan Stary
2020-08-28  4:54                                                 ` Jan Stary
2020-08-28  9:13                                                 ` Måns Rullgård
2020-08-24 12:19                                 ` Måns Rullgård
2020-08-24 14:43                                   ` Jan Stary
2020-08-24 14:59                                     ` Måns Rullgård
2020-08-27  9:44                                       ` Jan Stary
2020-08-27 11:08                                         ` Måns Rullgård
2020-08-24 17:22                                     ` Måns Rullgård
2020-08-22 15:43                     ` Jan Stary
2020-08-22 15:48                       ` Jan Stary
2020-08-24 12:18                   ` Jan Stary
2020-08-24 12:27                     ` Måns Rullgård
2020-08-24 13:16                       ` Jan Stary
2020-08-24 14:15                         ` Måns Rullgård
2020-08-27  9:52                           ` Jan Stary
2020-08-27 11:10                             ` Måns Rullgård
  -- strict thread matches above, loose matches on Subject: below --
2020-08-21  8:52 Jan Stary
2020-08-21 10:36 ` Måns Rullgård
2020-08-21 12:22   ` Jan Stary
2020-08-21 12:27     ` Måns Rullgård
2020-08-27  9:21 Jan Stary
2020-08-27 10:54 ` Jan Stary
2020-08-27 11:52 ` Måns Rullgård
2020-08-27 12:40   ` Jan Stary
2020-08-27 12:44     ` Jan Stary
2020-08-27 12:45     ` Måns Rullgård
2020-08-27 17:45 ` Jan Stary
2020-08-27 18:59   ` Måns Rullgård
2020-08-27 19:51     ` Jan Stary
2020-08-27 20:01 ` Jan Stary
2020-08-28  9:21   ` Måns Rullgård
2020-08-28  9:35     ` Jan Stary
2020-08-28  9:43       ` Måns Rullgård

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-list from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

  List information: https://lists.sourceforge.net/lists/listinfo/sox-devel

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20200821140458.GA48586@www.stare.cz \
    --to=sox-devel@lists.sourceforge.net \
    --cc=mans@mansr.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
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).