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
next prev parent 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).