Skip to content
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

make setup fails with /usr/bin/env: ‘bash\r’: No such file or directory #6201

Closed
TahaTesser opened this issue Aug 26, 2021 · 4 comments
Closed
Labels
infra.structure Relates to the tools that create docs.flutter.dev p3-low Valid but not urgent concern. Resolve when possible. Encourage upvote to surface. st.invalid Issue not relevant to repo or self-resolved

Comments

@TahaTesser
Copy link
Member

TahaTesser commented Aug 26, 2021

Describe the problem

When I set up docker image on Windows
make setup fails by 14/16

logs
Taha@Taha-PC MINGW64 ~/Code/cloned/website (master)
$ make setup
make clean
make[1]: Entering directory 'C:/Users/Taha/Code/cloned/website'
rm -rf _site .jekyll* src/.jekyll* *.log tmp example.g .dart_tool
make[1]: Leaving directory 'C:/Users/Taha/Code/cloned/website'
docker-compose build site
#1 [internal] load build definition from Dockerfile
#1 sha256:f561a8fef6cfe35823face857bde797a1c76b658b443e9a5e51c415e341effaa
#1 transferring dockerfile: 32B done
#1 DONE 0.0s

#2 [internal] load .dockerignore
#2 sha256:c72d804e4519ff661ddd75b049e656be79bd43eec3dfe302228373b4b39b99ec
#2 transferring context: 34B done
#2 DONE 0.0s

#3 [internal] load metadata for docker.io/library/ruby:3-buster
#3 sha256:df66394f91f845e96404cd0f06a808649ad0177e61733aadd5eff962a00ff40c
#3 DONE 0.7s

#4 [dev  1/16] FROM docker.io/library/ruby:3-buster@sha256:a24fb0a5dbbc2a2d1d87ca47ae3eeb643e9427da4a7849541aa0a0f2308860d1
#4 sha256:b342b20b19a5bc2536e1a5e0626480811f405d94d1c612f4ba98fc114893a587
#4 DONE 0.0s

#9 [internal] load build context
#9 sha256:e1fb1ff79debb986bfd813370f9ffc6d910279c0105814502da22139a8f8bb6f
#9 transferring context: 1.71MB 2.6s done
#9 DONE 2.7s

#5 [dev  2/16] RUN curl -sL https://deb.nodesource.com/setup_15.x -o node_setup.sh &&       bash node_setup.sh 1> /dev/null
#5 sha256:ab5bdd1089285e5a5d29d04522279a4b5aca7b0cfb131a063bbc3c1c454cd55e
#5 CACHED

#8 [dev  5/16] RUN gem install bundler
#8 sha256:84e9c48ade5f0d6a31d994076ec7f10afcce96929923779c57846bbc37d9416e
#8 CACHED

#14 [dev 10/16] RUN npm install
#14 sha256:e17d572aa1c70c5af26b9681145b61e28dd524a9fcfc19bdc0b857698e636c48
#14 CACHED

#16 [dev 12/16] RUN git submodule update --init --recursive
#16 sha256:98ff659eda107fabb38431b7324201b679395eef43d0c69a791d7838d9ebeb78
#16 CACHED

#6 [dev  3/16] RUN apt-get update && apt-get install -yq --no-install-recommends       build-essential       vim       git       diffutils       xdg-user-dirs       nodejs &&     rm -rf /var/lib/apt/lists/*
#6 sha256:d0b633dd5a50c0eb6c9768657ed61c01a1fd22372acea96c7b8e656d6a2bfb32
#6 CACHED

#13 [dev  9/16] RUN npm install -g npm firebase-tools superstatic
#13 sha256:4601b8ab9c0f191ce38bf9b893bb4edcfc5ccfd9d2d75373288a88d0f0851fea
#13 CACHED

#11 [dev  7/16] RUN bundle install
#11 sha256:4ab1ca9dbf3aa086d3ed0804f8ff38a9c20874d4e059eb48210a09bb376feeb2
#11 CACHED

#7 [dev  4/16] WORKDIR /app
#7 sha256:1ce5acde85d5dde7b2236d88b87873fcaf355993bd85b4e03a745623cd811020
#7 CACHED

#12 [dev  8/16] COPY package.json .
#12 sha256:04b84ce574ab66b81cef40e21b3e6ea99d639a2968dae9df5dc847affe9ac950
#12 CACHED

#10 [dev  6/16] COPY Gemfile .
#10 sha256:9ec258b9b44530261643c746929b9d08bd90094dda958ff60399fa891f04e9b2
#10 CACHED

#15 [dev 11/16] COPY ./ ./
#15 sha256:2448443432b590d2d1cdc393df7996cbd822a0c1fc1d894e01d46aacd6f247cd
#15 CACHED

#17 [dev 13/16] RUN if [[ -z  && "" != "stable" ]]; then       cd flutter;       git fetch;       git remote set-branches origin ;       git fetch --depth 1 origin ;       git checkout  --;       git pull;     fi
#17 sha256:763d2975dba933f49686fd2b928fd0299e4ada28ac1b12cf578ffe2e2fd03598
#17 CACHED

#18 [dev 14/16] RUN flutter doctor --suppress-analytics --quiet
#18 sha256:8bb2163916e27c2522bc25b7c21fc4dd0227ffd8e27db45f920db2810252ff5c
#18 0.378 /usr/bin/env: ‘bash\r’: No such file or directory
#18 ERROR: executor failed running [/bin/sh -c flutter doctor --suppress-analytics --quiet]: exit code: 127
------
 > [dev 14/16] RUN flutter doctor --suppress-analytics --quiet:
------
failed to solve: rpc error: code = Unknown desc = executor failed running [/bin/sh -c flutter doctor --suppress-analytics --quiet]: exit code: 127
make: *** [Makefile:52: setup] Error 17

Taha@Taha-PC MINGW64 ~/Code/cloned/website (master)

I also tried cd into website/flutter/bin and execute flutter doctor -v & flutter doctor --suppress-analytics --quiet but didn't help

Taha@Taha-PC MINGW64 ~/Code/cloned/website/flutter/bin ((2.2.3))
$ ./flutter doctor --suppress-analytics --quiet
Doctor summary (to see all details, run flutter doctor -v):
[√] Flutter (Channel unknown, 2.2.3, on Microsoft Windows [Version 10.0.22000.160], locale en-US)
[√] Android toolchain - develop for Android devices (Android SDK version 31.0.0)
[√] Chrome - develop for the web
[√] Visual Studio - develop for Windows (Visual Studio Community 2019 16.11.1)
[!] Android Studio (not installed)
[√] VS Code (version 1.59.1)
[√] Connected device (5 available)

! Doctor found issues in 1 category.

Taha@Taha-PC MINGW64 ~/Code/cloned/website/flutter/bin ((2.2.3))

I am using Git bash

PS C:\Users\Taha> make --version
GNU Make 4.3
Built for Windows32
Copyright (C) 1988-2020 Free Software Foundation, Inc.
License GPLv3+: GNU GPL version 3 or later <http://gnu.org/licenses/gpl.html>
This is free software: you are free to change and redistribute it.
There is NO WARRANTY, to the extent permitted by law.
PS C:\Users\Taha>
PS C:\Users\Taha> choco --version
0.10.15
PS C:\Users\Taha>
PS C:\Users\Taha> docker --version
Docker version 20.10.8, build 3967b7d
PS C:\Users\Taha>

Expected fix

Being to able to setup and serve website locally

Additional context

No response

@TahaTesser TahaTesser added the infra.structure Relates to the tools that create docs.flutter.dev label Aug 26, 2021
@TahaTesser
Copy link
Member Author

TahaTesser commented Aug 26, 2021

cc: @sfshaza2

@TahaTesser TahaTesser changed the title make setup fails in docker image make setup fails with /usr/bin/env: ‘bash\r’: No such file or directory Aug 26, 2021
@TahaTesser
Copy link
Member Author

I will try inside WSL tonight

@TahaTesser TahaTesser added infra.structure Relates to the tools that create docs.flutter.dev p3-low Valid but not urgent concern. Resolve when possible. Encourage upvote to surface. and removed infra.structure Relates to the tools that create docs.flutter.dev labels Aug 27, 2021
@TahaTesser
Copy link
Member Author

Closing for this now, maybe I was doing something incorrect.

@TahaTesser TahaTesser added the st.invalid Issue not relevant to repo or self-resolved label Oct 5, 2021
@skytomo221
Copy link

I recommend to execute git config --global core.autocrlf input
before git clone --recurse-submodules https://github.com/<username-or-flutter>/website.git
for Windows user.

parlough added a commit that referenced this issue Sep 16, 2023
_Description of what this PR is changing or adding, and why:_
The PR removes the `flutter` repo as a submodule to avoid conflicts when
building the website on different platforms. It was first introduced in
#1257 but with no explanation so we cannot track back what's the
purpose, but removing this should not cause difficulties in maintaining
the website.

### Additional info

- The build process is slower than the previous one because the checkout
step needs an extra 3~5 minutes, depending on the network condition. See
[now](https://github.com/flutter/website/actions/runs/5985413443/job/16237673762?pr=9307)
and
[previous](https://github.com/flutter/website/actions/runs/5983398408/job/16237673076?pr=9298)
checks.

_Issues fixed (partially) by this PR (if any):_
- #6201
- #6443
- #8272

## Presubmit checklist

- [x] This PR doesn’t contain automatically generated corrections
(Grammarly or similar).
- [x] This PR follows the [Google Developer Documentation Style
Guidelines](https://developers.google.com/style) — for example, it
doesn’t use _i.e._ or _e.g._, and it avoids _I_ and _we_ (first person).
- [x] This PR uses [semantic line
breaks](https://github.com/dart-lang/site-shared/blob/main/doc/writing-for-dart-and-flutter-websites.md#semantic-line-breaks)
of 80 characters or fewer.

---------

Co-authored-by: Anthony Sansone <[email protected]>
Co-authored-by: Parker Lougheed <[email protected]>
@AlexV525 AlexV525 linked a pull request Sep 17, 2023 that will close this issue
3 tasks
atsansone added a commit to atsansone/website that referenced this issue Sep 19, 2023
_Description of what this PR is changing or adding, and why:_
The PR removes the `flutter` repo as a submodule to avoid conflicts when
building the website on different platforms. It was first introduced in
flutter#1257 but with no explanation so we cannot track back what's the
purpose, but removing this should not cause difficulties in maintaining
the website.

### Additional info

- The build process is slower than the previous one because the checkout
step needs an extra 3~5 minutes, depending on the network condition. See
[now](https://github.com/flutter/website/actions/runs/5985413443/job/16237673762?pr=9307)
and
[previous](https://github.com/flutter/website/actions/runs/5983398408/job/16237673076?pr=9298)
checks.

_Issues fixed (partially) by this PR (if any):_
- flutter#6201
- flutter#6443
- flutter#8272

## Presubmit checklist

- [x] This PR doesn’t contain automatically generated corrections
(Grammarly or similar).
- [x] This PR follows the [Google Developer Documentation Style
Guidelines](https://developers.google.com/style) — for example, it
doesn’t use _i.e._ or _e.g._, and it avoids _I_ and _we_ (first person).
- [x] This PR uses [semantic line
breaks](https://github.com/dart-lang/site-shared/blob/main/doc/writing-for-dart-and-flutter-websites.md#semantic-line-breaks)
of 80 characters or fewer.

---------

Co-authored-by: Anthony Sansone <[email protected]>
Co-authored-by: Parker Lougheed <[email protected]>
atsansone added a commit that referenced this issue Sep 21, 2023
_Description of what this PR is changing or adding, and why:_
The PR removes the `flutter` repo as a submodule to avoid conflicts when
building the website on different platforms. It was first introduced in
#1257 but with no explanation so we cannot track back what's the
purpose, but removing this should not cause difficulties in maintaining
the website.

### Additional info

- The build process is slower than the previous one because the checkout
step needs an extra 3~5 minutes, depending on the network condition. See
[now](https://github.com/flutter/website/actions/runs/5985413443/job/16237673762?pr=9307)
and
[previous](https://github.com/flutter/website/actions/runs/5983398408/job/16237673076?pr=9298)
checks.

_Issues fixed (partially) by this PR (if any):_
- #6201
- #6443
- #8272

## Presubmit checklist

- [x] This PR doesn’t contain automatically generated corrections
(Grammarly or similar).
- [x] This PR follows the [Google Developer Documentation Style
Guidelines](https://developers.google.com/style) — for example, it
doesn’t use _i.e._ or _e.g._, and it avoids _I_ and _we_ (first person).
- [x] This PR uses [semantic line
breaks](https://github.com/dart-lang/site-shared/blob/main/doc/writing-for-dart-and-flutter-websites.md#semantic-line-breaks)
of 80 characters or fewer.

---------

Co-authored-by: Anthony Sansone <[email protected]>
Co-authored-by: Parker Lougheed <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
infra.structure Relates to the tools that create docs.flutter.dev p3-low Valid but not urgent concern. Resolve when possible. Encourage upvote to surface. st.invalid Issue not relevant to repo or self-resolved
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants