<html>
  <head>
    <meta http-equiv="Content-Type" content="text/html; charset=UTF-8">
  </head>
  <body>
    Hi,<br>
    <br>
    <div class="moz-cite-prefix">On 2022-07-24 10:44, r. a. schmied
      wrote:<br>
    </div>
    <blockquote type="cite" cite="mid:62DD8518.3090603@flash.net">On
      07/22/22 09:30, Ralph Little wrote:
      <br>
      <blockquote type="cite">Hi,
        <br>
        <br>
        On 2022-07-22 08:56, Ralph Little wrote:
        <br>
        <blockquote type="cite">Hi,
          <br>
          <br>
          On 2022-07-21 12:19, r. a. schmied wrote:
          <br>
          <blockquote type="cite">saners
            <br>
            <br>
            the missing '/' on line ending umax
            <br>
            (see rcsdiff -uBitwb configure.ac below)
            <br>
            might be why the umax_pp always
            <br>
            builds even with an envvar like
            <br>
            export BACKENDS='genesys net test'
            <br>
            passed to configure.
            <br>
            <br>
            yet to determine 1) why the backends
            <br>
            umax1220u v4l xerox_mfp p5
            <br>
            *do not get* built along with umax_pp and
            <br>
            2) if the added '/' will have any actual effect.
            <br>
            <br>
            i doubt this will have desired effect, as the
            <br>
            Makefile(s) do have 'BACKENDS =  genesys net test'
            <br>
            and do not have umax_pp (or any other backend)
            <br>
            in any obviously related envvar/macro.
            <br>
            <br>
            same missing '/' and build situation exists for
            <br>
            sane-backends-1.0.32.86-911be
            <br>
            and
            <br>
            sane-backends-1.0.28
            <br>
            <br>
            <br>
            ras
            <br>
            <br>
            --- configure.ac    2022/07/21 17:45:34    1.1
            <br>
            +++ configure.ac    2022/07/21 17:45:51
            <br>
            @@ -671,7 +671,7 @@
            <br>
                     mustek_usb mustek_usb2 nec net niash pie pieusb
            pint \
            <br>
                     pixma plustek plustek_pp qcam ricoh ricoh2 rts8891
            s9036 \
            <br>
                     sceptre sharp sm3600 sm3840 snapscan sp15c st400 \
            <br>
            -        stv680 tamarack teco1 teco2 teco3 test u12 umax
            <br>
            +        stv680 tamarack teco1 teco2 teco3 test u12 umax \
            <br>
                     umax_pp umax1220u v4l xerox_mfp p5"
            <br>
            <br>
             # If user specifies backends manually then cause configure
            <br>
            <br>
          </blockquote>
          OK, I looked into this a little bit.
          <br>
          <br>
          The issue is actually caused by the tools/ directory which
          builds a
          <br>
          tool called umax_pp.
          <br>
          This tool has a dependency on umax_pp_low which is why,
          despite the
          <br>
          umax_pp backend being disabled, the umax_pp_low.c source is
          built.
          <br>
          There doesn't appear to be a configured way to disable this
          being
          <br>
          built so I will add something.
          <br>
          <br>
          Thanks for raising this issue.
          <br>
          <br>
          Cheers,
          <br>
          Ralph
          <br>
          <br>
        </blockquote>
        OK, more detail on this.
        <br>
        <br>
        The umax_pp tool in the tools directory is always built.
        However, if you
        <br>
        disable the umax_pp backend then it merely sets the umax_pp tool
        to not
        <br>
        install, yet it is still built.
        <br>
        I think it is fair to say that if you are disabling the umax_pp
        backend,
        <br>
        then you are likely not interested in the umax_pp tool so I have
        removed
        <br>
        the build/noinstall option.
        <br>
        <br>
        I have added a check-in that if you disable the umax_pp backend,
        the
        <br>
        tool is not built.
        <br>
        <br>
        Cheers,
        <br>
        Ralph
        <br>
        <br>
        <br>
      </blockquote>
      <br>
      aloha ralph and interested saners
      <br>
      <br>
      ralph your analysis from a black-box view looks correct.
      <br>
      i've got sane-backends-1.0.27 installed and tools/umax_pp
      <br>
      was not installed, but tools/umax_pp was compiled and linked.
      <br>
      <br>
      for both sane-backends-1.0.28 and sane-backends-1.0.32.86-911be
      <br>
      tools/umax_pp was compiled and linked.  neither of these versions
      <br>
      have been installed yet [well 1.0.32.86-911be does not work, and
      <br>
      while 1.0.28 is better than 1.0.27 i keep messing with 1.0.28
      <br>
      to see if i can get it 'better' or at least more to my liking.
      <br>
      <br>
      [ backend issues with lide 200 and the contrast and brightness
      settings
      <br>
        and frontend issues [scanimage] with options and arguments that
      <br>
        as a gray-beard command liner (and horrible typist) drive me
      crazy ]
      <br>
      <br>
      is the 'check-in' patch simple enough for an email transmittal?
      <br>
      not critical, now that i understand the reason i can stop
      wondering
      <br>
      why.
      <br>
      <br>
      ras
      <br>
      <br>
    </blockquote>
    <br>
    Sure, the change is very simple.<br>
    <br>
    Look at the top of tools/Makefile.am and remove the following lines
    14 and 15:<br>
    <br>
    <div data-testid="left-content" class="diff-td line_content
      with-coverage left-side old"><span id="LC14" class="line"
        lang="make"><span class="k">else</span></span>
    </div>
    <div data-testid="left-content" class="diff-td line_content
      with-coverage left-side old"><span id="LC15" class="line"
        lang="make"><span class="nv">noinst_PROGRAMS</span> <span
          class="o">+=</span> umax_pp</span><br>
      <br>
      Then configure and build again. I don't think that you would need
      to do autogen.sh again although it wouldn't hurt.<br>
      <br>
      Cheers,<br>
      Ralph<br>
    </div>
  </body>
</html>