Bug 720374 - Files "condor_router_history", "condor_router_rm" and "condor_router_q" are not valid executable files for Windows
Summary: Files "condor_router_history", "condor_router_rm" and "condor_router_q" are n...
Keywords:
Status: CLOSED ERRATA
Alias: None
Product: Red Hat Enterprise MRG
Classification: Red Hat
Component: condor
Version: Development
Hardware: Unspecified
OS: Windows
low
medium
Target Milestone: 2.0.1
: ---
Assignee: Timothy St. Clair
QA Contact: MRG Quality Engineering
URL:
Whiteboard:
Depends On:
Blocks: 723887
TreeView+ depends on / blocked
 
Reported: 2011-07-11 14:32 UTC by Petra Svobodová
Modified: 2011-09-07 16:45 UTC (History)
5 users (show)

Fixed In Version: condor-7.6.3-0.3
Doc Type: Bug Fix
Doc Text:
Certain utility scripts did not end in a suffix, which is required for executables on Windows. These scripts are no longer included in the MSI installer.
Clone Of:
Environment:
Last Closed: 2011-09-07 16:45:40 UTC
Target Upstream Version:
Embargoed:


Attachments (Terms of Use)


Links
System ID Private Priority Status Summary Last Updated
Red Hat Bugzilla 712529 0 high CLOSED condor_router_* scripts incorrect in 7_6 series 2021-02-22 00:41:40 UTC
Red Hat Product Errata RHSA-2011:1249 0 normal SHIPPED_LIVE Moderate: Red Hat Enterprise MRG Grid 2.0 security, bug fix and enhancement update 2011-09-07 16:40:45 UTC

Internal Links: 712529

Description Petra Svobodová 2011-07-11 14:32:46 UTC
Description of problem:
The files  "condor_router_history", "condor_router_rm" and "condor_router_q" have no suffix; they are not valid executable files for Windows. 
In the last stable condor version (condor-7.6.1-0.11) these files are executable  ("condor_router_history.exe", "condor_router_rm.exe" and "condor_router_q.exe").

Version-Release number of selected component (if applicable):
condor-7.6.3-0.1.msi

How reproducible:
100%

Steps to Reproduce:
1. Install the .msi package and compare content of the "bin" folder with the stable version (condor-7.6.1-0.11).
  
Actual results:
Files "condor_router_history", "condor_router_rm" and "condor_router_q" have no suffix

Expected results:
The files should have suffixes; they should be executable files

Additional info:

Comment 1 Timothy St. Clair 2011-07-11 15:09:22 UTC
updated to add their correct .py extension in distribution see dependent BZ information

Comment 2 Petra Svobodová 2011-07-13 08:07:14 UTC
Files "condor_router_rm" and "condor_router_q" are shell scripts. 

Would you possibly to look at these files, please?

Comment 3 Timothy St. Clair 2011-07-13 13:29:30 UTC
thanks for the update, apparently I was a bit quick on the draw.  

I've modified to .sh extension on install.  You will need to setup file associations with cygwin in order to run.

Comment 4 Timothy St. Clair 2011-07-13 19:25:21 UTC
    Technical note added. If any revisions are required, please edit the "Technical Notes" field
    accordingly. All revisions will be proofread by the Engineering Content Services team.
    
    New Contents:
C: Build modifications exposed a flaw in distribution, when fixed causes issues when trying to run on windows
C: Without a file extension users will have to determine the correct interpreter and invoke by hand
F: Add correct extension during installation prior to packaging
R: Installs should have the correct file extensions.

Comment 6 Timothy St. Clair 2011-07-14 15:22:38 UTC
Kicking back to assigned b/c we have decided to remove these tools from the windows build and they have never worked.

Comment 7 Timothy St. Clair 2011-07-14 15:22:38 UTC
Deleted Technical Notes Contents.

Old Contents:
C: Build modifications exposed a flaw in distribution, when fixed causes issues when trying to run on windows
C: Without a file extension users will have to determine the correct interpreter and invoke by hand
F: Add correct extension during installation prior to packaging
R: Installs should have the correct file extensions.

Comment 8 Timothy St. Clair 2011-07-14 15:32:48 UTC
Modified upstream for conditional build flag removal.  Thus is will not be in our next revision.

Comment 10 Timothy St. Clair 2011-07-25 16:32:05 UTC
    Technical note added. If any revisions are required, please edit the "Technical Notes" field
    accordingly. All revisions will be proofread by the Engineering Content Services team.
    
    New Contents:
C: Build modifications exposed a flaw in distribution to distro executables when they are scripts.
C: Tools did not work
F: Remove the tools from distribution on windows.
R: Tools are no longer distributed in the .msi package.

Comment 11 Martin Kudlej 2011-07-27 12:14:57 UTC
There are still condor_router_* are in condor-win-7.6.3-0.2.msi and there is not any other newer package. -->ASSIGNED

Comment 13 Martin Kudlej 2011-07-28 10:40:22 UTC
Tested on Windows 2003/2008/7 x x86_64/i386 and Windows XP i386 and Windows 2008 R2 x86_64 with condor-win-7.6.3-0.3.msi and condor_router_* are not there. -->VERIFIED

Comment 14 Douglas Silas 2011-08-08 14:00:14 UTC
    Technical note updated. If any revisions are required, please edit the "Technical Notes" field
    accordingly. All revisions will be proofread by the Engineering Content Services team.
    
    Diffed Contents:
@@ -1,4 +1 @@
-C: Build modifications exposed a flaw in distribution to distro executables when they are scripts.
+Certain utility scripts did not end in a suffix, which is required for executables on Windows.  These scripts are no longer included in the MSI installer.-C: Tools did not work
-F: Remove the tools from distribution on windows.
-R: Tools are no longer distributed in the .msi package.

Comment 15 errata-xmlrpc 2011-09-07 16:45:40 UTC
An advisory has been issued which should help the problem
described in this bug report. This report is therefore being
closed with a resolution of ERRATA. For more information
on therefore solution and/or where to find the updated files,
please follow the link below. You may reopen this bug report
if the solution does not work for you.

http://rhn.redhat.com/errata/RHSA-2011-1249.html


Note You need to log in before you can comment on or make changes to this bug.