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

/tmp is not mounted to %TEMP% #143

Closed
orgads opened this issue May 10, 2015 · 21 comments
Closed

/tmp is not mounted to %TEMP% #143

orgads opened this issue May 10, 2015 · 21 comments

Comments

@orgads
Copy link

orgads commented May 10, 2015

It used to be in MsysGit.

@PhilipOakley
Copy link

There has been quite some discussion around the tmp/TEMP directories you may want to look at in the various issue/PR discussions. e.g. #123

How is this affecting you (do you have an example to help with a Minimal, Complete and Verifiable Example ;-)

@PhilipOakley
Copy link

For searching, try putting "%TEMP% in:title,body" in the filter box of the issues page (you (& I) learn something every day) or "%TEMP% in:title,body,comment" etc. See the https://help.github.com/articles/searching-issues/ page for some examples

@dscho
Copy link
Member

dscho commented May 11, 2015

Since %TEMP% is user-dependent, I am not quite sure how that would be encoded properly in an fstab (the file where drive mappings are specified for Cygwin/MSys2). There might already be an option for that, or it will require appropriate changes in msys-2.0.dll -- accompanied by the corresponding change in the git-extra package.

@orgads if you are interested in tackling this, please have a look at this description how to build the MSys2 runtime ("msys-2.0.dll").

@dscho
Copy link
Member

dscho commented May 20, 2015

@orgads I was hoping to interest you in this issue, so I implemented a very first version of a topic branch to help with this issue, but I need your help now: could you have a look, test, and see whether you can do something about the output of mount.exe when called without arguments?

@orgads
Copy link
Author

orgads commented May 21, 2015

@dscho Thanks for your efforts. I'm sorry for not being responsive lately. I was loaded with work.

I'll try to get to this on next Monday.

@dscho
Copy link
Member

dscho commented May 22, 2015

@orgads thanks. Please note that my current plan is to release Git for Windows on Monday.

@dscho
Copy link
Member

dscho commented May 25, 2015

@orgads out of curiosity: did you work on this yet?

@orgads
Copy link
Author

orgads commented May 25, 2015

I won't have time today. Hopefully tomorrow. Let's leave it for the next
release.
בתאריך 25 במאי 2015 3:21 אחה״צ,‏ "dscho" notifications@github.com כתב:

@orgads https://github.com/orgads out of curiosity: did you work on
this yet?


Reply to this email directly or view it on GitHub
#143 (comment)
.

@dscho
Copy link
Member

dscho commented May 25, 2015

I just merged an updated version of git-for-windows/msys2-runtime#9. It lists /tmp/ correctly in the mount output.

@dscho dscho closed this as completed May 25, 2015
@orgads
Copy link
Author

orgads commented May 25, 2015

Great!

I actually tried to build msys2-runtime but couldn't get it to compile (some fpermissive errors...)

@dscho
Copy link
Member

dscho commented May 25, 2015

You have to build it from the MSys2 shell, i.e. start C:\git-sdk-64\msys2_shell.bat. Otherwise you are in a MINGW instead of an MSYS build environment.

@orgads
Copy link
Author

orgads commented May 25, 2015

That's what I did. It starts building fine, but after a few minutes I'm getting many errors such as:

c++wrap -pipe -O2 -g -ggdb -fno-rtti -fno-exceptions -fno-use-cxa-atexit -Wall -Wstrict-aliasing -Wwrite-strings -fno-common -pipe -fbuiltin -fmessage-length=0 -MMD -Werror -fmerge-constants -ftracer -mcmodel=small -c -o dlfcn.o /usr/src/MSYS2-packages/msys2-runtime/src/msys2-runtime/winsup/cygwin/dlfcn.cc
In file included from ./globals.h:6:0,
                 from /usr/src/MSYS2-packages/msys2-runtime/src/msys2-runtime/winsup/cygwin/winsup.h:311,
                 from /usr/src/MSYS2-packages/msys2-runtime/src/msys2-runtime/winsup/cygwin/debug.cc:11:
/usr/src/MSYS2-packages/msys2-runtime/src/msys2-runtime/winsup/cygwin/winsup.h:192:45: error: default argument given for parameter 1 of 'void close_all_files(bool)' [-fpermissive]
 void __stdcall close_all_files (bool = false);

@dscho
Copy link
Member

dscho commented May 25, 2015

Oh... slaps-his-head. To build the msys2-runtime, actually, any MSys2 package, you need to call

pacman -Sy base-devel binutils

once, to initialize the build system.

@orgads
Copy link
Author

orgads commented May 25, 2015

@dscho Thanks. I already did that. I'm building by running makepkg in msys2-runtime directory in MSYS2-packages. Should I use something else?

@dscho
Copy link
Member

dscho commented May 25, 2015

no, that's correct. Maybe your PATH is wrong? What does which c++wrap.exe say?

@orgads
Copy link
Author

orgads commented May 26, 2015

which: no c++wrap.exe in (...my path...)

@dscho
Copy link
Member

dscho commented May 26, 2015

Hmm, neither is it in the PATH here. So that means that something went wrong at configure time.

@dscho
Copy link
Member

dscho commented May 26, 2015

Okay, so this is the output here:

$ echo $MSYSTEM
MSYS
$ uname -a
MSYS_NT-6.3 dscho 2.1.2(0.288/5/3) 2015-05-25 15:48 x86_64 Msys
$ cd  /usr/src/MSYS2-packages/msys2-runtime/src/build-x86_64-pc-msys/x86_64-pc-msys/winsup/cygwin
$ rm -f dlfcn.o && make dflcn.o
c++wrap -pipe -O2 -g -ggdb -fno-rtti -fno-exceptions -fno-use-cxa-atexit -Wall -Wstrict-aliasing -Wwrite-strings -fno-common -pipe -fbuiltin -fmessage-length=0 -MMD -Werror -fmerge-constants -ftracer -mcmodel=small -c -o dlfcn.o /usr/src/MSYS2-packages/msys2-runtime/src/msys2-runtime/winsup/cygwin/dlfcn.cc

And it works correctly here, obviously... Maybe you can spot differences in your setup?

@orgads
Copy link
Author

orgads commented May 26, 2015

It's exactly the same.

gcc --version
gcc (GCC) 4.9.2

Yours?

@dscho
Copy link
Member

dscho commented May 26, 2015

$ gcc -v
Using built-in specs.
COLLECT_GCC=gcc
COLLECT_LTO_WRAPPER=/usr/lib/gcc/x86_64-pc-msys/4.9.2/lto-wrapper.exe
Target: x86_64-pc-msys
Configured with: /build2/gcc/src/gcc-4.9.2/configure --build=x86_64-pc-msys --prefix=/usr --libexecdir=/usr/lib --enable-bootstrap --enable-shared --enable-shared-libgcc --enable-static --enable-version-specific-runtime-libs --with-arch=x86-64 --disable-multilib --with-tune=generic --enable-__cxa_atexit --with-dwarf2 --enable-languages=c,c++,fortran,lto --enable-graphite --enable-threads=posix --enable-libatomic --enable-libgomp --disable-libitm --enable-libquadmath --enable-libquadmath-support --enable-libssp --disable-win32-registry --disable-symvers --with-gnu-ld --with-gnu-as --disable-isl-version-check --enable-checking=release --without-libiconv-prefix --without-libintl-prefix --with-system-zlib
Thread model: posix
gcc version 4.9.2 (GCC)

@orgads
Copy link
Author

orgads commented Aug 17, 2015

Finally found the reason for the compilation error: git-for-windows/msys2-runtime#12

jeffhostetler pushed a commit to jeffhostetler/git that referenced this issue Jun 4, 2019
Replace `git multi-pack-index expire/repack` with upstream v6
jeffhostetler pushed a commit to jeffhostetler/git that referenced this issue Jun 13, 2019
Replace `git multi-pack-index expire/repack` with upstream v6
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants