Perl 6 - the future is here, just unevenly distributed

IRC log for #native, 2017-05-11

| Channels | #native index | Today | | Search | Google Search | Plain-Text | summary

All times shown according to UTC.

Time Nick Message
00:54 felliott joined #native
03:40 sivoais joined #native
03:44 willthechill joined #native
05:31 sivoais joined #native
09:53 mohawk_pts joined #native
10:21 mohawk_pts joined #native
12:07 mohawk_pts_ joined #native
13:46 dj joined #native
14:35 dj I'd like to bundle a patch with my distribution.  Where should it live?  the Alien::Build::Manual::FAQ: example of %{.install.patch} resolves to an empty string
14:36 dj Hit return too soon.  I'm using Alien::Build with an alienfile.
14:42 mohawk_pts_ plicease, ^
14:54 dj To sidestep that issue, I've decided to do it directly in Perl. The sub is passed an Alien::Build object, but there's no attribute which provides the directory containing the extracted archive.
14:56 plicease current directory is the extracted archive.
14:56 plicease it isn't a property because there may be multiple extractions
14:56 dj ok.  thanks
14:56 plicease %{.install.patch} should be replaced by ./patch in your dist root
14:57 plicease do you have an example on git I can see if this is a bug in AB or something else?
14:58 plicease well.  almost %{.install.patch} should be an absolute path.
14:58 dj I'm still getting an empty string.  I'll work something up and put it up on github
14:59 plicease thanks.
15:07 dj now it works.  of course.
15:07 plicease oh dear :(
15:07 dj no clue why it didn't before
15:08 plicease it may be a typo.  I think trying to interpolate undefined properties should probably throw an exception intead of interpolating to ''
15:09 plicease it is probably early enough to make this change I don't think it should break anything that isn't already broken.
15:12 dj it may have been. I don't recall changing anything.  now on to why my DESTDIR hack isn't working for this particular library.  I presume DESTDIR is on by default for autoconf?  some autoconf makefiles are still handcrafted (as is this one) and don't support DESTDIR
15:13 plicease yes.  https://metacpan.org/pod/distribution/Alien-Build/lib/Alien/Build/Manual/FAQ.pod#autoconf-like
15:13 plicease use Build::MSYS instead in that case.
15:14 plicease oh I see an embarassing typo
15:15 dj are you setting DESTDIR via the environment or via the make command line?
15:16 plicease environment.
15:17 plicease I think you can also turn it off from the alienfile via meta->prop->{destdir} = 0;
15:17 plicease as long as you do it after loading the build::Autoconf plugin.
15:18 plicease nevermind, I don't think the plugin will work without DESTDIR
15:18 plicease use Build::MSYS
15:19 dj this makefile dates back to AFS and has an INSTALL_ROOT which provides the same functionality as DESTDIR, but explicitly sets INSTALL_ROOT=. I've renamed INSTALL_ROOT to DESTDIR, but I think it can only be overridden via a command line setting if it is actually set in the makefile.  checking now.
15:19 plicease Ah I remember AFS.
15:20 plicease (from my IBM days)
15:20 plicease Is that common?  I wonder if it makes sense to support a configurable name for DESTDIR
15:21 dj I knew of it only from afar.  Here's what GNU make's manual says: "an explicit assignment in the makefile, or with a command argument, overrides the environment"
15:21 dj it might be safer to specify DESTDIR on the command line
15:22 dj I've found it to be very uncommon.
15:29 plicease I can see that argument.  Doing it in the command line requires the alienfile author to add more stuff to get interpolated
15:29 plicease and it feels complicated enough
15:30 dj AHA.  %{.install.patch} is empty if I use af, but not if I go the perl Makefile.PL route.
15:30 plicease the autoconf manual also says the makefile targets other than install and uninstall shouldn't
15:31 plicease so setting DESTDIR shouldn't harm the other make targets (in theory)
15:31 plicease ah.
15:31 plicease so bug in af
15:32 dj as soon as I relearn my regular expressions I'll upload this to github so you can have a definitive test case.
15:32 plicease cool.
15:33 plicease Please add any details you think pertinent here: https://github.com/plicease/App-af/issues/3
15:45 dj will do.  see https://github.com/djerius/Alien-XPA
16:09 mohawk_pts__ joined #native
17:58 willthechill joined #native

| Channels | #native index | Today | | Search | Google Search | Plain-Text | summary