-
Notifications
You must be signed in to change notification settings - Fork 92
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
Allow for a SYSGPORT configuration statement #505
Comments
Why? What's the problem? Why would you want/need to use a separate port? |
Currently, if you have a SYSG defined, the first 3270 connection automatically gets assigned to SYSG. Then any subsequent connections are assigned to 3270 devices. But this may not be desirable. On real hardware, SYSG is started on demand and one could have lots of 3270 devices without SYSG.
|
Only if your configuration file is coded to behave that way.
Understood. Which is why Hercules supports the |
P.S. It is preferred that you not respond/reply to GitHub Issues via email When you reply directly via their web page, I can make minor edits to your reply so it is more readable (prettier) by editing the fonts being used, formatting log messages, etc. When you reply via email however, I cannot edit your reply (GitHub does not allow it), so oftentimes it is much harder (more difficult) to read. It is up to you whether or not you want to take the time to reply via their web page or continue to reply via email, but it is generally preferable that you reply directly via their web page instead. Thanks (*) GitHub does not support formatting of, nor attaching files to, email replies, making it impossible for me to fix the formatting of a person's reply for readability or receive the file that was requested from them. Thank you for understanding. |
Thank you, Fish. The problem may be with my 3270 emulator. I can't specify a groupname or a device address on the hostname. I may be wrong, but it looks like the doc implies that not all emulators support that. Currently I just comment out the SYSG and only put it back in when needed. Again, I thought this is a nice to have, but if you feel its not needed, you can close it out. |
What emulator are you using?
It certainly has merit, I'll give you that. And it should certainly be easy enough to implement too IMO, so I think we should probably keep this request open. Maybe someone out there will be able to implement it for you before I do. (*) Any takers? (*) I'd do it myself but I'm kind of busy with the "COD" thing right now, so if I do it it'll have to wait until later. Sorry. But maybe somebody else can do it before I can? Hello? Anyone? |
I did try a few different 3270 emulators to see if I can accomplish this using a group name. I found only the wc3270 command line allows for entering a groupname@hostname:port. But the session wizard does not allow for a '@' in the hostname. |
Aaron (@arfineman), Closed by commit 6dee8ff. Please give it a try and tell me what you think. Thanks. |
Hi Fish, |
As am I.
What's the problem? Building Hercules from source is actually pretty easy IMO. Of course, I'm a developer too, and so do it all the time, which is why I find it to be easy. But even for the average non-developer Windows user it's not really that difficult. In fact, Bill Lewis (i.e. @wrljet, a fellow Hercules developer) has created a very nice, easy to use Windows Power Shell helper script called "Hercules Helper" that totally automates the entire process, making doing so dirt simple. You just enter a simple command, press enter, and in a few minutes, Voilà! You've got a working version of Hercules on your system. Of course you have to clone his repository first, which requires having Try it! I think you'll be glad you did. (Bill? Does your script support building the "develop" branch?) |
"develop" branch is the default. You can download the Hercules-Helper-Windows repo as a zip file from GitHub, without git. I'm here, and all ears, to help anyone who wants it. Bill |
That's good to hear!
(Doh!) Of course. How silly of me. |
I have also just created a macOS Homebrew installation formula. |
Correction, "develop" is not the default. |
Thank you, Bill. I downloaded the zip file. But neither my W7 or W10 can open the 'README.md' to start the process. Am I doing something wrong, or do I need to download something to read that file. I do have Office 2016 installed. |
Please ignore the previous message. Looks like if I associate .md file extension to Wordpad, it opens fine. My bad. |
Aaron,
If you scroll down on the GitHub page past the file list, it displays the I don't know how to view those with their formatting on Windows. Bill |
What do you mean by "cannot open"? Did you "unblock" the downloaded .zip file before you unzipped it, as Bill's web site explains in Step 2?
This is standard Windows behavior:
I believe ".md" files are treated as browser files, just like ".htm" or ".html" files are, and thus either your browser or Windows itself may be purposely preventing you from opening them since it considers them "untrustworthy" since they came from a foreign system. You need to manually unblock the file before attempting to open it. As far as displaying the If you so search the web there are even "online" web pages where you can upload or paste your markdown file/code into a text box and it will display the rendered markdown code for you. (Markdown files are just simple, specially formatted text files, similar to HTML files that can also be opened in Notepad, but which when opened in a browser are parsed and displayed in a much more user friendly manner.) |
But all of that (what I've written in my previous comment above) is only if you want to view on your own system the information that already exists on Bill's web site: Visit the above web page and scroll down maybe one page, and you will see exactly the same information that is in the |
You need to use a browser extension, such as Markdown Viewer. |
You shouldn't need to do that! You should (IMHO) leave the file associated with your browser, just like it was. Opening and properly displaying .md files is not that hard to do once you know how. (And once you UNBLOCK them beforehand, of course!) |
I don't do Chrome. |
What do you do? |
Firefox, and when that doesn't work (which is more and more common on websites these days), the new MSFT Edge. |
|
But I already did that the first time. And again here: PS C:\users\asus\desktop\hrc\hrc> set-executionpolicy remotesigned
Execution Policy Change
The execution policy helps protect you from scripts that you do not trust. Changing the execution policy might expose
you to the security risks described in the about_Execution_Policies help topic. Do you want to change the execution
policy?
[Y] Yes [N] No [S] Suspend [?] Help (default is "Y"): y
PS C:\users\asus\desktop\hrc\hrc> .\hyperion-buildall
File C:\users\asus\desktop\hrc\hrc\hyperion-buildall.ps1 cannot be loaded. The file C:\users\asus\desktop\hrc\hrc\hyper
ion-buildall.ps1 is not digitally signed. The script will not execute on the system. Please see "get-help about_signing
" for more details..
At line:1 char:20
+ .\hyperion-buildall <<<<
+ CategoryInfo : NotSpecified: (:) [], PSSecurityException
+ FullyQualifiedErrorId : RuntimeException
PS C:\users\asus\desktop\hrc\hrc> |
OK Let's start with, what Windows are you on, and what version of PowerShell is it? With that info I'll fire up a fresh copy in a VM. Bill |
One more question: did you close and reopen the PowerShell window after doing |
OK. Making progress. I exited PS and came back in. Now it is complaining about a PS C:\users\hp\desktop\hrc\hrc> .\hyperion-buildall.ps1
Transcript started, output file is .\hercules-helper-20221018_18-56-49.log
Options:
C:\users\hp\desktop\hrc\hrc\hyperion-buildall.ps1 : Error: -BuildDir parameter is missing
At line:1 char:1
+ .\hyperion-buildall.ps1
+ ~~~~~~~~~~~~~~~~~~~~~~~
+ CategoryInfo : NotSpecified: (:) [Write-Error], WriteErrorException
+ FullyQualifiedErrorId : Microsoft.PowerShell.Commands.WriteErrorException,hyperion-buildall.ps1
Transcript stopped, output file is C:\users\hp\desktop\hrc\hrc\hercules-helper-20221018_18-56-49.log PS C:\users\hp\desktop\hrc\hrc> .\hyperion-buildall.ps1 -gitbranch develop
Transcript started, output file is .\hercules-helper-20221018_18-58-58.log
Options:
C:\users\hp\desktop\hrc\hrc\hyperion-buildall.ps1 : Error: -BuildDir parameter is missing
At line:1 char:1
+ .\hyperion-buildall.ps1 -gitbranch develop
+ ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
+ CategoryInfo : NotSpecified: (:) [Write-Error], WriteErrorException
+ FullyQualifiedErrorId : Microsoft.PowerShell.Commands.WriteErrorException,hyperion-buildall.ps1
Transcript stopped, output file is C:\users\hp\desktop\hrc\hrc\hercules-helper-20221018_18-58-58.log
PS C:\users\hp\desktop\hrc\hrc> |
It's all documented in Bill's README! Refer to the instructions for Step 4: Decide where (what directory) you want Hercules to be built, and decide if you prefer to use Visual Studio 2017, 2019, or 2022, and then specify those options (as well as the -GitBranch option too) in your command, e.g:
|
Thank you. I misunderstood. I thought that script does all those steps for you. Success! This script is unbelievable! Build is complete. Now I can pickup the latest and greatest without having to wait for a new official release. Now I can test the Thanks so much to Bill and Fish! |
Glad to hear it! Aaron, if you don't mind, would you find the log file it created, the one with the newest timestamp, zip it and send to me. |
Will do. This was done on my home PC as will take care of it this evening when I get home. Thanks again for putting this together for dummies like me. I had tried it several times before on my own without success. |
Hi Fish, I tested the SYSGPORT as a configuration statement and also as a command and it works perfectly. I see that you set the default port number to 3278. That works great for me. I added a SYSGPORT 3278 to my config file regardless. I should point out however that this may be problematic for an existing user of SYSG that is used to the old port 3270, as this change may not be considered backward compatible. This is strictly your call if you want the default to be 3270 or 3278. |
Unfortunately the Your point is valid however: introducing new unexpected behavior is rarely a Good Thing. I suppose I could tweak the new code a tiny bit to support specifying e.g. Let me think about it... |
Log attached.
…On Wed, Oct 19, 2022 at 10:55 AM Bill Lewis ***@***.***> wrote:
Glad to hear it!
Aaron, if you don't mind, would you find the log file it created, the one
with the newest timestamp, zip it and send to me.
I'd like to review it. The filename will be along the lines of
hercules-helper-20220920_18-26-13.log.
—
Reply to this email directly, view it on GitHub
<#505 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/APLI2226PXQWBCJV7XN34NDWEADW3ANCNFSM6AAAAAAQYADSYU>
.
You are receiving this because you were mentioned.Message ID:
***@***.***>
|
Um, no it isn't. This is yet another reason why I prefer that people respond directly to GitHub Issues and not via email. |
Sorry, I replied this time, expecting it directly to go to Bill. Here is the attachment. |
That would only occur if you manually changed the email "To:" field to his email address. If you don't do that, the email goes "To:" GitHub, which, as explained, doesn't support attachments. |
Fish, |
This *should* provide original behavior when no SYSGPORT statement is given, while providing the new behavior when a SYSGPORT statement *is* given [with a valid port number(*)]. This *should* now PROPERLY close GitHub Issue #505. -------------- (*) Same as current CNSLPORT behavior, if an invalid port number is specified, a default value is used (which for SYSGPORT is 3278).
Done! Aaron? (@arfineman) Please rebuild and re-test. The original Hercules behavior (before any of my changes) should be restored now when a SYSGPORT statement is not specified (or is specified as "NO"). The new behavior is only triggered whenever a SYSGPORT statement is specified with a valid port number. This should prevent "unexpected new behavior" for our existing users while still allowing them to take advantage of the new behavior should they wish to. Thanks. I am closing this issue as completed again. This time hopefully for good! |
Aaron, Hercules-Helper will not re-clone or update your local SDL-Hercules-390 git repo. So to update and rebuild:
The Visual Studio installation should have left you with a command line Then rerun your original Bill |
Fish, |
Fish, The latest commit 66ff70a , related to this SYSGPORT, is busted (on Linux). Hercules won't connect to the 3270 console with Bill |
That's odd. The logic is platform independent and should work identically on *Nix as it does on Windows. I promise you I did test it on Windows, and it worked just fine! Aaron, who also uses Windows, also reported that it worked just fine for him too, so why the frick it's failing on *nix I have no freaking idea. (sigh) Let me look into it an get back to you... |
(oops!) How the frick did that happen? Sorry! Should be fixed now. |
Thanks. Is that ( 44dc0f8) really the title/comment you want in that commit? |
Sure! Why not? It was just a fix of a silly typo. No need to get into details IMO. |
It would be nice to have a SYSGPORT statement to distinguish a SYSG port from a 3270 port. If not specified, it would default to the current CNSLPORT.
The text was updated successfully, but these errors were encountered: