Skip to content

Commit

Permalink
Revert firedrake-install location, to avoid breaking firedrake-update
Browse files Browse the repository at this point in the history
  • Loading branch information
connorjward committed Nov 25, 2024
1 parent 8bcb96f commit 867337f
Show file tree
Hide file tree
Showing 5 changed files with 7 additions and 6 deletions.
2 changes: 1 addition & 1 deletion .github/workflows/build.yml
Original file line number Diff line number Diff line change
Expand Up @@ -59,7 +59,7 @@ jobs:
cd ..
# Linting should ignore unquoted shell variable $COMPLEX
# shellcheck disable=SC2086
./firedrake/firedrake/scripts/firedrake-install \
./firedrake/scripts/firedrake-install \
$COMPLEX \
--honour-petsc-dir \
--mpicc="$MPICH_DIR"/mpicc \
Expand Down
2 changes: 1 addition & 1 deletion .github/workflows/zenodo-canary.yml
Original file line number Diff line number Diff line change
Expand Up @@ -25,7 +25,7 @@ jobs:
pip install requests packaging
- name: Zenodo API canary
run: |
python firedrake/scripts/firedrake-install --test-doi-resolution
python scripts/firedrake-install --test-doi-resolution
- name: Upload log
uses: actions/upload-artifact@v4
if: failure()
Expand Down
1 change: 1 addition & 0 deletions Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -17,6 +17,7 @@ lint:
@python -m flake8 $(FLAKE8_FORMAT) firedrake
@echo " Linting firedrake scripts"
@python -m flake8 $(FLAKE8_FORMAT) firedrake/scripts --filename=*
@python -m flake8 $(FLAKE8_FORMAT) scripts --filename=*
@echo " Linting firedrake tests"
@python -m flake8 $(FLAKE8_FORMAT) tests
@echo " Linting PyOP2"
Expand Down
4 changes: 2 additions & 2 deletions docs/source/download.rst
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,7 @@ Obtaining Firedrake

Firedrake is installed using its install script::

curl -O https://raw.githubusercontent.com/firedrakeproject/firedrake/master/firedrake/scripts/firedrake-install
curl -O https://raw.githubusercontent.com/firedrakeproject/firedrake/master/scripts/firedrake-install

In the simplest cases, such as on a Mac with Homebrew installed or on
an Ubuntu workstation on which the user has sudo acccess, the user can simply run::
Expand Down Expand Up @@ -232,7 +232,7 @@ You should now be able to run ``firedrake-update``.
Installing Firedrake with pip (experimental, Linux only)
--------------------------------------------------------

Firedrake is working towards having support for binary installs. As such there is experimental support for installing Firedrake using ``pip``, avoiding the need for a ``firedrake-install`` script. At present only Linux is tested using this install method.
Firedrake has experimental support for installing using ``pip``, avoiding the need for the ``firedrake-install`` script. At present only Linux is tested using this install method.

Requirements
~~~~~~~~~~~~
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -1397,7 +1397,7 @@ def quit(message):

def build_update_script():
log.info("Creating firedrake-update script.")
with open("firedrake/firedrake/scripts/firedrake-install", "r") as f:
with open("firedrake/scripts/firedrake-install", "r") as f:
update_script = f.read()

try:
Expand All @@ -1410,7 +1410,7 @@ def build_update_script():


if args.rebuild_script:
os.chdir(os.path.dirname(os.path.realpath(__file__)) + ("/../../.."))
os.chdir(os.path.dirname(os.path.realpath(__file__)) + ("/../.."))

build_update_script()

Expand Down

0 comments on commit 867337f

Please sign in to comment.