Use 'type' instead of 'which' which is standard unix

Resolves https://bugzilla.redhat.com/show_bug.cgi?id=2196701
cockpit-project · May 11, 2023 · 3a0391d · 3a0391d
1 parent e430ed1
commit 3a0391d
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 3 deletions.
diff --git a/src/app.jsx b/src/app.jsx
@@ -188,7 +188,7 @@ class AppActive extends React.Component {
         if (this.props.error)
             this.onAddErrorNotification({ text: _("Failed to fetch some resources"), detail: this.props.error });
 
-        cockpit.spawn(['which', 'virt-install'], { err: 'ignore' })
+        cockpit.script('type virt-install', { err: 'ignore' })
                 .then(() => {
                     this.setState({ virtInstallAvailable: true });
                     cockpit.spawn(['virt-install', '--install=?'], { err: 'ignore' })

diff --git a/src/components/vm/overview/vmOverviewCard.jsx b/src/components/vm/overview/vmOverviewCard.jsx
@@ -63,7 +63,7 @@ class VmOverviewCard extends React.Component {
     }
 
     componentDidMount() {
-        cockpit.spawn(['which', 'virt-xml'], { err: 'ignore' })
+        cockpit.script('type virt-xml', { err: 'ignore' })
                 .then(() => {
                     this.setState({ virtXMLAvailable: true });
                 }, () => this.setState({ virtXMLAvailable: false }));

diff --git a/src/components/vm/vmActions.jsx b/src/components/vm/vmActions.jsx
@@ -64,7 +64,7 @@ const VmActions = ({ vm, onAddErrorNotification, isDetailsPage }) => {
     const [virtCloneAvailable, setVirtCloneAvailable] = useState(false);
 
     useEffect(() => {
-        cockpit.spawn(['which', 'virt-clone'], { err: 'ignore' })
+        cockpit.script('type virt-clone', { err: 'ignore' })
                 .then(() => setVirtCloneAvailable(true));
     }, []);