Importing the xonsh shell into the FreeBSD ports tree.
AbandonedPublic
Actions

Authored by matthew on Nov 8 2016, 5:08 PM.

Details

Reviewers

larafercue_gmail.com
mat

Group Reviewers

Ports Committers

Summary

The xonsh shell is a Python-ish shell that can be
usefull for a developer, this patch is meant to importing the
software in the ports tree.

Diff Detail

Repository

rP FreeBSD ports repository

Lint

No Lint Coverage

Unit

No Test Coverage

Build Status

Buildable 5893
Build 6180: arc lint + arc unit

Event Timeline

larafercue_gmail.com updated this revision to Diff 22092.Nov 8 2016, 5:08 PM

larafercue_gmail.com retitled this revision from to Importing the xonsh shell into the FreeBSD ports tree..

larafercue_gmail.com updated this object.

larafercue_gmail.com edited the test plan for this revision. (Show Details)

Herald added subscribers: Contributor Reviewers (ports), mat. · View Herald TranscriptNov 8 2016, 5:08 PM

mat requested changes to this revision.Nov 8 2016, 5:59 PM

mat added a reviewer: mat.

mat added inline comments.

shells/xonsh/Makefile
3–5	There are tabs missing after the = here.
9	This must be removed.
12	This is the default and can be removed.
14	the build and run are implied and can be removed.
shells/xonsh/pkg-descr
6	This should be: WWW: http://xon.sh

This revision now requires changes to proceed.Nov 8 2016, 5:59 PM

Modifying the patch with the changes indicated by mat.

Removing the local changes to the xorg-server

mat added inline comments.Nov 23 2016, 4:26 PM

shells/xonsh/Makefile
17	empty line

Removing the empty line at the end of the Makefile as indicated by mat.

Revision accepted, updated and setting the flag done to the revision comments.

larafercue_gmail.com added a reviewer: Ports Committers.Jul 10 2017, 8:03 AM

This revision now requires changes to proceed.Jul 10 2017, 8:03 AM

Is there a PR for this?

In D8475#238955, @matthew wrote:

Is there a PR for this?

I could not find any searching for it in bugzilla, so perhaps I forgot to open it. Should I open it?

In D8475#238957, @roberfern_gmail.com wrote:

In D8475#238955, @matthew wrote:

Is there a PR for this?

I could not find any searching for it in bugzilla, so perhaps I forgot to open it. Should I open it?