Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

x3270: init at 3.4ga9 #11080

Closed
wants to merge 2 commits into from
Closed

x3270: init at 3.4ga9 #11080

wants to merge 2 commits into from

Conversation

musteresel
Copy link
Contributor

I added a nix expression to build the x3270 IBM 3270 terminal emulator, to be found there:

http://x3270.bgp.nu/index.html

I packaged the current stable version (3.4ga9 stable (23. October 2015), according to their downloads page). Though I pull the source code from the git repository instead of the provided tar balls. A small patch to fix the interpreter of an utility bash script was needed.

Actually this package now provides multiple binaries. One could add support to only build a subset of these binaries (and thus removing tcl, x dependencies). Also there are no man pages, because the respective install target fails and I don't have the time to investigate further.

As for testing I'm successfully running the applications on my laptop (NixOS on x86_64), using them to participate in the IBM Master the Mainframe contest. No further testing than this was done, though.

Please let me know if there's something that needs to be changed (this is my first to be contributed package). This is a second pull request, the first #11031 targeted the release branch and I couldn't get it to target the master branch.


patches = [ ./repair-interpreter.patch ];

meta = {
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Could you add meta.license?

x3270 is an IBM 3270 terminal emulator for the X Window System. It comes
with some other - related - tools, such as a ncurses version or a
version intended for use with scripts. These tools are currently
unconditionally built, this is a possible point of improvement. Also the
man pages unfortunately don't install with this nix expression.

Version 3.4ga9 is the current (23. Oct 2015) stable version of this
software.
@musteresel
Copy link
Contributor Author

@pSub I added the licence as well as corrected the description. I amended the "main commit" with these changes. The homepage of the tool does provide source tarballs of the current stable release. I thought that using the tag from the repository would be the more stable approach, but I added another commit using fetchurl and this tarball instead. Though I cannot test it locally on my machine atm.

@rickynils
Copy link
Member

@musteresel It seems your latest commit (2d79062) didn't work: https://travis-ci.org/NixOS/nixpkgs/builds/91654817. Also, the commit description should ideally include the package it changes.

@rasendubi
Copy link
Member

(triage) @musteresel do you expect to work on this futher?

@grahamc
Copy link
Member

grahamc commented Feb 13, 2017

Closing. Can re-open if you would like :)

@grahamc grahamc closed this Feb 13, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants