public inbox for cygwin-apps@cygwin.com
 help / color / mirror / Atom feed
From: Marcos Vives Del Sol <socram8888@gmail.com>
To: cygwin-apps@cygwin.com
Subject: Re: [ITP] libsass v3.3.6
Date: Tue, 07 Jun 2016 09:36:00 -0000	[thread overview]
Message-ID: <CABKfh3zMEN_O3fvaqtxRZjWCw8eLspS2W3X9XrJqYrMaA3Ndrw@mail.gmail.com> (raw)
In-Reply-To: <878tyiz1o6.fsf@Rainer.invalid>

2016-06-06 19:07 GMT+02:00 Achim Gratz <Stromeko@nexgo.de>:
> That file is created by configure, so patching it in anyway is still
> useless.
No. The file created by configure is version.h.

VERSION file later is employed by the version.sh script:
-----------------
if test "x$LIBSASS_VERSION" = "x"; then
  LIBSASS_VERSION=`git describe --abbrev=4 --dirty --always --tags 2>/dev/null`
fi
if test "x$LIBSASS_VERSION" = "x"; then
  LIBSASS_VERSION=`cat VERSION 2>/dev/null`
fi
if test "x$LIBSASS_VERSION" = "x"; then
  LIBSASS_VERSION="[na]"
fi
echo $LIBSASS_VERSION
-----------------
Former will fail because it's not a git repository, so it falls back
to reading from VERSION file, or else it will use "[na]".

This script is called using m4_esyscmd_s from configure.ac line 6:
-----------------
AC_INIT([libsass], m4_esyscmd_s([./version.sh]), [support@moovweb.com])
-----------------
to set the library version at compile time. Then configure uses it to
generate the version.h file.

The patch is supposed to create that file at
./libsass-3.3.6-1.i686/src/VERSION, but for some reason cygport
creates it at ./libsass-3.3.6-1.i686/src/libsass-3.3.6/src/libsass-3.3.6/VERSION,
so version.sh can't read it and reports "[na]" anyway. Any insight on
why the patch created by cygport itself isn't working?

-- 
Marcos Vives Del Sol
https://github.com/socram8888
miniLock: NscWdW8fh2E5SBoRtrnXWrDwmQmwaR5awMKE78tQW1Ffx

  reply	other threads:[~2016-06-07  9:36 UTC|newest]

Thread overview: 12+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-06-02 15:10 Marcos Vives Del Sol
2016-06-06 10:09 ` Marco Atzeri
2016-06-06 16:46   ` Marcos Vives Del Sol
2016-06-06 17:07     ` Achim Gratz
2016-06-07  9:36       ` Marcos Vives Del Sol [this message]
2016-06-07  9:49         ` Achim Gratz
2016-06-07 10:02           ` Marco Atzeri
2016-06-07 22:18             ` Marcos Vives Del Sol
2016-06-08  5:46               ` Achim Gratz
2016-06-08  8:02                 ` Marcos Vives Del Sol
2016-06-08 18:55                   ` Achim Gratz
2016-06-09  9:37               ` Marco Atzeri

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-all from there: mbox

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

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

  git send-email \
    --in-reply-to=CABKfh3zMEN_O3fvaqtxRZjWCw8eLspS2W3X9XrJqYrMaA3Ndrw@mail.gmail.com \
    --to=socram8888@gmail.com \
    --cc=cygwin-apps@cygwin.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.
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).