Re: Internship on Improve the user experience for the "guix package" command line tool (Outreachy)

[Gábor Boskovits]

2018-03-21 18:03 GMT+01:00 Vijayalakshmi Vedantham <address@hidden>:

Thank you so much to everyone who replied. This project truly has the kindest and most polite people.

I wanted to know what kind of system most people work on. My ubuntu is giving me Locale errors so I was thinking of creating a virtual environment and working on that. Do you think that will work out?

Most of the time I use a laptop with Ubuntu as a host system, and GuixSD running in a KVM/qemu virtual machine. It works fine for me.

 

Also, has the patch been applied yet?

Yes, I see that the patch has been applied already.

 

Thanks,

Vijayalakshmi

On Wed, Mar 21, 2018 at 2:05 PM, Efraim Flashner <address@hidden> wrote:

On Tue, Mar 20, 2018 at 08:45:02PM +0530, Vijayalakshmi Vedantham wrote:
> Hi,
>
> I'm really sorry about the effort you had to put into this patch. I'll try
> not to do it again.
>
> I had to append “.zip” because Pypi didn’t have a “.tar.gz” file for the
> > sources, so the uri
> > field now is:
> >
> >    (pypi-uri "logwrap" version ".zip")
> >
>
> Did you do this because only .zip is available here (
> https://pypi.python.org/pypi/logwrap#downloads)?
>

$ guix import pypi logwrap

Starting download of /tmp/guix-file.BdwXcD
>From https://pypi.python.org/packages/b3/17/c7d450ce6a1ce82e145852895509fddc9468225d2aa312a772bb9c188a73/logwrap-3.2.1.zip...
 …3.2.1.zip  241KiB                   869KiB/s 00:00 [##################] 100.0%

In this case the importer grabbed a zip file, so I guess it falls into
tribal knowledge, definately something that should be documented better.

>
> >
> > I also noticed that the sources include files that were generated with
> > Cython.  Instead of reusing those, we build them from source.  Luckily
> > all we have to do in this case is to add “python-cython” to the
> > native-inputs field.
> >
>
> Can I know how you knew this?

For files that are generated by Cython they generally have a line at the
top that reads: "Generated by Cython 0.x.y" or something along those
lines. If you unzip the downloaded source and run 'grep Cython . -R'
inside the folder it'll show any files generated by Cython.

>
>
> > Finally, the tests. At first the tests wouldn’t run.  So I looked up the
> > error message online and found that I need to use “python-pytest-runner”
> > in addition to “python-pytest”.  This allows the tests to run up to a
> > point until it wants to do coverage tests.  For those I needed to add
> > “python-pytest-cov”.
> >
> > I changed the description, because I think it wasn’t quite correct.
> >
>
> Again, I'm sorry.

No worries, the test suite wasn't so easy. The release tarball/zip
didn't include good directions for running the test suite, I had to go
to the actual repository and see how the tests were run with 'tox' and
then try recreating the commands; I wasn't successful.

As far as the description goes, it's my least favorite part :).

>
>
>      Have you been able to build Guix already and try building the package
> using “./pre-inst-env guix build”?
>
> No, I tried last night but I faced some issues, I'll try again tonight.
>
> Thanks,
> Vijayalakshmi

--
Efraim Flashner   <address@hidden>   אפרים פלשנר
GPG key = A28B F40C 3E55 1372 662D  14F7 41AA E7DC CA3D 8351
Confidentiality cannot be guaranteed on emails sent or received unencrypted