Hi,
you are probably busy so I'd like to ask if this requires further
changes or if it is good to go. Someone else can review it too.
Thanks
ng0 <ngillm...@runbox.com> writes:
> Leo Famulari <l...@famulari.name> writes:
>
>> [ Unknown signature status ]
>> On Sat, Sep 24, 2016 at 01:04:47PM +0000, ng0 wrote:
>>> I tried to get ssl working via:
>>> - patching include/config.h
>>> - just export the variable which was in include/config.h without
>>> patching it.
>>>
>>> but it seems like epic5 does something different which 'justworks™' in
>>> ircii. epic5 is based on ircii.
>>> If ssl is mandatory, I'll check their git repository and see if there
>>> have been any commits recently fixing this.
>>
>> In general, SSL / TLS support is not mandatory for packages that do
>> network traffic. But, users should not be misled into thinking that they
>> are using TLS when it doesn't work.
>>
>> So, what happens when you use this epic5 to connect to an IRC server
>> that uses TLS?
>>
>> Does it fail silently? That's not okay.
>
> No, it fails with (for me) cryptic failures like ircII used to do. You
> can try it out yourself by comparing ircii and epic5 while trying
> chat.freenode.net 6697
>
> unset IRCSERVER; epic5 chat.freenode.net -p 6697
> output:
>
> *** I can't find your mailbox.
> *** Performing DNS lookup for [chat.freenode.net] (server 0)
> *** DNS lookup for server 0 [chat.freenode.net] returned (15) addresses
> *** Connecting to server refnum 0 (chat.freenode.net), using address 1
> (164.132.77.237:6697)
> *** INFO -- unix_recv: read(3) failed: Connection reset by peer
> *** INFO -- new_io_event: fd 3 must be closed
> *** INFO -- dgets: fd [3] must be closed
> *** Connection closed from chat.freenode.net
>
>
> ... I assume that epic5 just isn't up to date or does handle something
> differently which ircii succeeds with. ircii added this
> functionality/variable:
> export IRCSERVER="SSLIRC/chat.freenode.net:6697"
> |
> so if you prepend "SSLIRC/" and use |
> :whatever-port-the-networks-ircs-port-is ----
> you get a connection to ircs, otherwise it fails.
> Best is to try this yourself for comparing outputs.
>
>>> Subject: [PATCH 1/2] licenses: Add epic.
>>>
>>> * guix/licenses.scm (epic): New variable.
>>
>> Does anything else use this license? If not, I would leave it out and
>> use non-copyleft for the epic5 package license.
>>
>>> Subject: [PATCH 2/2] gnu: Add epic5.
>>>
>>> * gnu/packages/irc.scm (epic5): New variable.
>>
>>> + (add-after 'unpack 'patch-bsdinstall
>>> + ;; We should include this in the build-system, bsdinstall is a
>>> standard.
>>> + ;; If we just remove /bin/ some part of the bsdinstall breaks.
>>> + (lambda _
>>> + (substitute* "bsdinstall"
>>> + (("/bin/strip") "strip")
>>> + (("/bin/cp") "cp")
>>> + (("/bin/chmod") "chmod")
>>> + (("/etc/chown") "chown")
>>> + (("/bin/chgrp") "chgrp")
>>> + (("/bin/mkdir") "mkdir")
>>> + (("/bin/rm") "rm")
>>> + (("/bin/mv") "mv")
>>> + (("/etc/") ""))))
>>
>> "/etc/" is not an executable path. Do we need to substitute it here?
>> What happens when you don't change that line?
>
> bsdinstalls has a reference to /etc/chmod there - if we leave /etc/ in,
> install fails.
>
>>> + (zero?
>>> + (system* "./configure"
>>> + (string-append "--prefix=" out)
>>> + "--with-ipv6" "--with-libarchive"
>>
>> What does it use libarchive for? What happens if we leave it out?
>
> I added it because of our "maximum features" goal.
>
> UPDATES:1473:*** News 01/03/2008 -- $info(o) values for libarchive, iconv
> support
> UPDATES:1474: If the binary supports libarchive, $info(o) will include 'r'.
> UPDATES:1509:*** News 11/29/2007 -- Support for ZIP files from libarchive
> UPDATES:1529: All of this is based on 'libarchive' being installed. You
> will need
> UPDATES:1530: to re-run configure in order to pick up libarchive support
> after you
>
> Details:
>
> *** News 01/03/2008 -- $info(o) values for libarchive, iconv support
> If the binary supports libarchive, $info(o) will include 'r'.
> If the binary supports iconv, $info(o) will include 'v'.
> Libarchive support is required to /load from a .zip file
> Iconv support is required to be able to do character set translation.
>
> *** News 11/29/2007 -- Support for ZIP files from libarchive
> Support for loading files from .zip files has been added. This first
> round of implementation just adds the raw ability, but it's not
> totaly ready to be used yet. You're welcome to start playing with
> it and reporting any problems you have.
>
> You can $open() a file for reading or /load it from a zip file:
> /load foo.zip/file
> and
> @fd = open(foo.zip/file R)
>
> If you /load a zip file, it will load the file ".ircrc" in the top
> level directory. This might be enahanced or changed in the future:
> /load foo.zip
> acts like
> /load foo.zip/.ircrc
>
> Some operations cannot be performed on zipped files, such as $fseek()
> and $frewind() and so forth. This might change in the future.
>
> All of this is based on 'libarchive' being installed. You will need
> to re-run configure in order to pick up libarchive support after you
> do a cvs update.
> --
> ng0
>
>
--
