Swordfish90 / cool-retro-term

A good looking terminal emulator which mimics the old cathode display...
22.34k stars 851 forks source link

Gentoo ebuild #76

Closed tgerczei closed 10 years ago

tgerczei commented 10 years ago

Hey! I pretty much fell in love with this at first glance. However, I'd like to be able to install it properly and would therefore like to have an ebuild for Gentoo available. I'll do it myself at some point later in time but as I've never done one from scratch before and won't have a suitable machine on my hands soon enough, it might take a while - so an experienced volunteer would be most welcome. :)

Swordfish90 commented 10 years ago

Thank you. I'm glad you like it. I've never used gentoo, so I can't help you here, but maybe someone else is interested in this and will help you.

tgerczei commented 10 years ago

I'm on it already. If it works, I'll contribute it.

tgerczei commented 10 years ago

Here it is; I've never done one before so I guess it sucks; nevertheless it did the trick for me. You will need the overlay "qt" ('emerge layman && layman -a qt) as well as likely a local overlay to place this ebuild in. I'd appreciate feed-back! cool-old-term_ebuild.tar | uploaded via ZenHub

annunaki2k2 commented 10 years ago

Hi tgerczei, Not sure if you've taken down your ebuild file, or if zenhub.io is having troubles, but I keep being greeted with an nginx 502 error when using that link. Any chance you can repost your ebuild? Or can you upload it to an overlay we can sync from with layman? Cheers,

tgerczei commented 10 years ago

Hi,

updated ebuild for a local overlay: https://www.dropbox.com/s/222yjlbgge09477/cool-old-term_ebuild.tgz?dl=0 I know it's lame, but I haven't set up an overlay yet.

annunaki2k2 commented 10 years ago

Thanks bud - just one minor alteration and it worked like a charm. Many thanks, and here is the alteration I had to make:

--- cool-retro-term-9999.ebuild.orig 2014-10-03 12:59:36.907140662 +0100 +++ cool-retro-term-9999.ebuild 2014-10-03 12:59:49.224140808 +0100 @@ -24,7 +24,7 @@ DEPEND="

=dev-qt/qtcore-5.3.1 =dev-qt/qtquickcontrols-5.3.1[widgets] ->=dev-qt/qtdeclarative-5.3.1 +>=dev-qt/qtdeclarative-5.3.1[localstorage] =dev-qt/qtgraphicaleffects-5.3.1 " src_compile() {

tgerczei commented 10 years ago

Oh, missed a flag :) Thanks for the hint and enjoy!

grondinm commented 10 years ago

Thanx for the ebuild

grondinm commented 10 years ago

so i'm sure this is something stupid but when i try to run it after installing with this ebuild this is the output

cool-retro-term loadAllColorSchemes file:///opt/cool-retro-term/app/qml/PreprocessedTerminal.qml:146: ReferenceError: shellProgram is not defined qml: Storing settings: {"bloom_quality":1,"fontIndexes":[0,0,0],"fontScaling":1,"fps":0,"frameReflections":true,"scanline_quality":3,"showMenubar":true,"show_terminal_size":true,"window_scaling":1} qml: Storing profile: {"ambient_light":0.2,"background_color":"#000000","bloom_strength":0.65,"brightness":0.5,"brightness_flickering":0.1,"chroma_color":0,"contrast":0.85,"fontIndex":0,"fontWidth":1,"font_color":"#ff8100","frames_index":1,"glowing_line_strength":0.2,"horizontal_sincronization":0.08,"jitter":0.18,"motion_blur":0.4,"noise_strength":0.1,"rasterization":0,"rgb_shift":0,"saturation_color":0,"screen_distortion":0.1,"windowOpacity":1}

tgerczei commented 10 years ago

I'm glad it proves useful! As for this issue, I have no clue unfortunately but something has likely changed upstream. Try cloning the repo manually and taking a look at the referenced file or better yet see if someone else has seen the same error by feeding that line to a search engine of your choice. I have no access to my system until the end of the month so my options are limited.

On 2014. október 14. 0:37:40 Marc Grondin notifications@github.com wrote:

so i'm sure this is something stupid but when i try to run it after installing with this ebuild this is the output

cool-retro-term loadAllColorSchemes file:///opt/cool-retro-term/app/qml/PreprocessedTerminal.qml:146: ReferenceError: shellProgram is not defined qml: Storing settings: {"bloom_quality":1,"fontIndexes":[0,0,0],"fontScaling":1,"fps":0,"frameReflections":true,"scanline_quality":3,"showMenubar":true,"show_terminal_size":true,"window_scaling":1} qml: Storing profile: {"ambient_light":0.2,"background_color":"#000000","bloom_strength":0.65,"brightness":0.5,"brightness_flickering":0.1,"chroma_color":0,"contrast":0.85,"fontIndex":0,"fontWidth":1,"font_color":"#ff8100","frames_index":1,"glowing_line_strength":0.2,"horizontal_sincronization":0.08,"jitter":0.18,"motion_blur":0.4,"noise_strength":0.1,"rasterization":0,"rgb_shift":0,"saturation_color":0,"screen_distortion":0.1,"windowOpacity":1}


Reply to this email directly or view it on GitHub: https://github.com/Swordfish90/cool-retro-term/issues/76#issuecomment-58965159

grondinm commented 10 years ago

Well i've come up empty...

This is the line it complains about

        // Retrieve the variable set in main.cpp if arguments are passed.
        if (shellProgram)
            ksession.setShellProgram(shellProgram);
        if (workdir)
            ksession.initialWorkingDirectory = workdir;

        ksession.startShellProgram();
        forceActiveFocus();
    }
}

if i look in main.cpp i see this

engine.rootContext()->setContextProperty("shellProgram", getNamedArgument(args, "--program"));

i'm not 100% sure what all this does i'm no programer...google only found this page...guess i'm left with my uncool modern term for now

tgerczei commented 10 years ago

Please give it another go. I've encountered the same issue and have added a patch to the ebuild that comments out the two variables I can't seem to find out how to populate. It did make it work for me although this is by no means what should have been done. :) The tarball has been updated accordingly.

Swordfish90 commented 10 years ago

Hi guys, I'm interested in why you are having issues with that undefined variable. Basically here I'm just setting a variable in main.cpp that is later read in the qml side. I can't reproduce the issue on my machines. Do you have only issues on the QML side or can you see any error also in the native code? Are you launching the program with or without arguments?

tgerczei commented 10 years ago

Hi Filippo,

Great that You picked this up! :) I'd also like to know why it behaves differently now but wanted to deliver Marc a quick temporary fix.

I have been wondering as well why it branches to the 'argument provided' part of the QML condition evaluation and initially sought to somehow provide the variable on the command line when I saw it exit referencing an undeclared variable but I failed to figure that out and admittedly didn't even try too hard. Please let me know what I'm doing wrong when I'm starting the application with "/usr/lib/qt5/bin/qmlscene -I /opt/cool-retro-term/imports /opt/cool-retro-term/app/qml/main.qml" - as this worked so far...! I'd be happy to fix my ebuild properly.

On Sat, 8 Nov 2014, Filippo Scognamiglio wrote:

Hi guys, I'm interested in why you are having issues with that undefined variable. Basically here I'm just setting a variable in main.cpp that is later read in the qml side. I can't reproduce the issue on my machines. Do you have only issues on the QML side or can you see any error also in the native code? Are you launching the program with or without arguments?

— Reply to this email directly or view it on GitHub.[AGBnlWG_9ah6-Ad8UCx-9-Dsauu_159tks5nLrp1gaJpZM4CVaT9.gif]

Swordfish90 commented 10 years ago

@tgerczei Ok, I got it. That's easy to solve. Launching the application with qmlscene completely bypass what I do on the native side (main.cpp), including setting those variables. Run the application from the binary executable ./cool-retro-term and everything should be fine. It should load much faster and should support all the arguments I've introduced.

tgerczei commented 10 years ago

Thanks, that cleared it up. I haven't even been building that target so far and never even knew... Package updated. Might make it a bit better later on.

tgerczei commented 9 years ago

Package is now sanitized and published via my own portage overlay. Please see README.md for instructions and report any issues! Thanks @Swordfish90 :>

happysmash27 commented 5 years ago

tgerczei: Your overlay is no longer working.

happysmash27 commented 5 years ago

Nevermind, I needed to update the overlay file from wget https://www.gerczei.eu/files/gerczei.xml -O /etc/layman/overlays/gerczei.xml.

tgerczei commented 5 years ago

@happysmash27 OK - I think this has also been mentioned in the README. Please let me know if that's incorrect or insufficient!

happysmash27 commented 4 years ago

It seems the ebuild is now using an outdated EAPI, giving errors whenever I run emerge:

 * ERROR: x11-terms/cool-retro-term-9999::gerczei failed (depend phase):
 *   qmake-utils.eclass: unsupported EAPI=5
 *
 * Call stack:
 *                     ebuild.sh, line 616:  Called source '/var/lib/layman/gerczei/x11-terms/cool-retro-term/cool-retro-term-9999.ebuild'
 *   cool-retro-term-9999.ebuild, line   7:  Called inherit 'git-r3' 'qmake-utils' 'eutils'
 *                     ebuild.sh, line 317:  Called __qa_source '/usr/portage/eclass/qmake-utils.eclass'
 *                     ebuild.sh, line 112:  Called source '/usr/portage/eclass/qmake-utils.eclass'
 *            qmake-utils.eclass, line  20:  Called die
 * The specific snippet of code:
 *   [[ ${EAPI:-0} == [012345] ]] && die "qmake-utils.eclass: unsupported EAPI=${EAPI:-0}"
 *
 * If you need support, post the output of `emerge --info '=x11-terms/cool-retro-term-9999::gerczei'`,
 * the complete build log and the output of `emerge -pqv '=x11-terms/cool-retro-term-9999::gerczei'`.
 * Working directory: '/usr/lib64/python3.6/site-packages'
 * S: '/var/tmp/portage/x11-terms/cool-retro-term-9999/work/cool-retro-term-9999'
tgerczei commented 4 years ago

Thank You for reporting! Yes, that's known and I have this issue open. That said, as per this pull request this package had been superseded. It hasn't been merged yet, though, so the official documentation wouldn't let you know right now. At any rate, please emerge "x11-terms/cool-retro-term-1.1.1::gentoo" instead. If you still need the bleeding edge -9999 ebuild, please tell me or just patch it via a pull request and I'll be happy to merge it.