llvm dev - Dec 2004 - [LLVMdev]Undefinedreferenceto`llvm::sys::CopyFile(llvm::sys::Pathconst&, llvm::sys::P

This is one of the reasons I'm getting rid of the "platform" link.
I'm
not sure what platform you're working on, but on Cygwin it causes
problems. Cygwin doesn't like .. through a soft link and generates
incorrect file names. I imagine this is what is happening to the
dependencies. In any event, the "platform" link will be gone before
the
new year. 

Sorry for all the confusion recently with lib/System. I'm nearly done
with it. After the consolidation into just "Unix" and
"Win32"
implementations is done, it will be left alone for a long time. We all
have better things to do than fuss with lib/System.

Reid.

On Fri, 2004-12-24 at 03:47, Henrik Bach wrote:
> Hi Reid,
> 
> Looking closer at the build process, I find that Path.cpp doesn't get 
> compiled, even if it is a newer version than its *.o file. That's
probaly
> the reason the error still exists.
> 
> I've zipped and attached the trace after the point the new Path.cpp was
> updated by cvs. It clearly shows that it newer gets compiled and now shows 
> that it is the dependencies in the makefile that is out of sync.
> 
> I'll try to remove the .o file and see what happens.
> 
> Yes, this time Path.cpp got compiled.
> 
> All this tells me that we don't have correct dependencies in place.
> 
> 
> Merry Christmas
> Henrik.
> 
> 
> ----Original Message Follows----
> From: "Henrik Bach" <henrik_bach_llvm at hotmail.com>
> Reply-To: LLVM Developers Mailing List <llvmdev at cs.uiuc.edu>
> To: llvmdev at cs.uiuc.edu
> Subject: Re: [LLVMdev] 
> Undefinedreferenceto`llvm::sys::CopyFile(llvm::sys::Path const&, 
> llvm::sys::P
> Date: Fri, 24 Dec 2004 00:48:26 +0100
> 
> Hi Reid,
> 
> The error still exists. However, looking on the function in Win32/Path.cpp:
> --------------
> void
> sys::CopyFile(const sys::Path &Dest, const sys::Path &Src) {
>   if (!::CopyFile(Src.c_str(), Dest.c_str(), false))
>     ThrowError("Can't copy '" + Src.toString() +
>                "' to '" + Dest.toString() +
"'");
> }
> --------------
> 
> I notice, that the function is living in the sys namespace. Shouldn't
it
> live in the Path class instead? A copy-past to fast ;-)
> 
> 
> Henrik.
> 
> _________________________________________________________________
> Log på MSN Messenger direkte fra nettet  http://webmessenger.msn.com/
> 
> _______________________________________________
> LLVM Developers mailing list
> LLVMdev at cs.uiuc.edu         http://llvm.cs.uiuc.edu
> http://mail.cs.uiuc.edu/mailman/listinfo/llvmdev
> 
> _________________________________________________________________
> Få alle de nye og sjove ikoner med MSN Messenger http://messenger.msn.dk/
> 
> _______________________________________________
> LLVM Developers mailing list
> LLVMdev at cs.uiuc.edu         http://llvm.cs.uiuc.edu
> http://mail.cs.uiuc.edu/mailman/listinfo/llvmdev
> 
> _________________________________________________________________
> Log på MSN Messenger direkte fra nettet  http://webmessenger.msn.com/
> << my_make.zip >>
> _______________________________________________
> LLVM Developers mailing list
> LLVMdev at cs.uiuc.edu         http://llvm.cs.uiuc.edu
> http://mail.cs.uiuc.edu/mailman/listinfo/llvmdev
> 
> _________________________________________________________________
> Del din verden med MSN Spaces http://spaces.msn.com
> 
> _______________________________________________
> LLVM Developers mailing list
> LLVMdev at cs.uiuc.edu         http://llvm.cs.uiuc.edu
> http://mail.cs.uiuc.edu/mailman/listinfo/llvmdev
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 189 bytes
Desc: This is a digitally signed message part
URL:
<http://lists.llvm.org/pipermail/llvm-dev/attachments/20041224/210c5006/attachment.sig>

And I had just fixed VC++ so that it *did* handle the dependencies with 
platform correctly...  This means that each file in System you change to 
eliminate the #include from platform will require a corresponding change 
to System.vproj.  I'd appreciate it if you got it over with all at once, 
otherwise VC++ support will be broken more often than not.

Reid Spencer wrote:
>This is one of the reasons I'm getting rid of the "platform"
link. I'm
>not sure what platform you're working on, but on Cygwin it causes
>problems. Cygwin doesn't like .. through a soft link and generates
>incorrect file names. I imagine this is what is happening to the
>dependencies. In any event, the "platform" link will be gone
before the
>new year. 
>
>Sorry for all the confusion recently with lib/System. I'm nearly done
>with it. After the consolidation into just "Unix" and
"Win32"
>implementations is done, it will be left alone for a long time. We all
>have better things to do than fuss with lib/System.
>
>Reid.
>
>On Fri, 2004-12-24 at 03:47, Henrik Bach wrote:
>  
>
>>Hi Reid,
>>
>>Looking closer at the build process, I find that Path.cpp doesn't
get
>>compiled, even if it is a newer version than its *.o file. That's
probaly
>>the reason the error still exists.
>>
>>I've zipped and attached the trace after the point the new Path.cpp
was
>>updated by cvs. It clearly shows that it newer gets compiled and now
shows
>>that it is the dependencies in the makefile that is out of sync.
>>
>>I'll try to remove the .o file and see what happens.
>>
>>Yes, this time Path.cpp got compiled.
>>
>>All this tells me that we don't have correct dependencies in place.
>>
>>
>>Merry Christmas
>>Henrik.
>>
>>
>>----Original Message Follows----
>>From: "Henrik Bach" <henrik_bach_llvm at hotmail.com>
>>Reply-To: LLVM Developers Mailing List <llvmdev at cs.uiuc.edu>
>>To: llvmdev at cs.uiuc.edu
>>Subject: Re: [LLVMdev] 
>>Undefinedreferenceto`llvm::sys::CopyFile(llvm::sys::Path const&, 
>>llvm::sys::P
>>Date: Fri, 24 Dec 2004 00:48:26 +0100
>>
>>Hi Reid,
>>
>>The error still exists. However, looking on the function in
Win32/Path.cpp:
>>--------------
>>void
>>sys::CopyFile(const sys::Path &Dest, const sys::Path &Src) {
>>  if (!::CopyFile(Src.c_str(), Dest.c_str(), false))
>>    ThrowError("Can't copy '" + Src.toString() +
>>               "' to '" + Dest.toString() +
"'");
>>}
>>--------------
>>
>>I notice, that the function is living in the sys namespace.
Shouldn't it
>>live in the Path class instead? A copy-past to fast ;-)
>>
>>
>>Henrik.
>>
>>_________________________________________________________________
>>Log på MSN Messenger direkte fra nettet  http://webmessenger.msn.com/
>>
>>_______________________________________________
>>LLVM Developers mailing list
>>LLVMdev at cs.uiuc.edu         http://llvm.cs.uiuc.edu
>>http://mail.cs.uiuc.edu/mailman/listinfo/llvmdev
>>
>>_________________________________________________________________
>>Få alle de nye og sjove ikoner med MSN Messenger
http://messenger.msn.dk/
>>
>>_______________________________________________
>>LLVM Developers mailing list
>>LLVMdev at cs.uiuc.edu         http://llvm.cs.uiuc.edu
>>http://mail.cs.uiuc.edu/mailman/listinfo/llvmdev
>>
>>_________________________________________________________________
>>Log på MSN Messenger direkte fra nettet  http://webmessenger.msn.com/
>><< my_make.zip >>
>>_______________________________________________
>>LLVM Developers mailing list
>>LLVMdev at cs.uiuc.edu         http://llvm.cs.uiuc.edu
>>http://mail.cs.uiuc.edu/mailman/listinfo/llvmdev
>>
>>_________________________________________________________________
>>Del din verden med MSN Spaces http://spaces.msn.com
>>
>>_______________________________________________
>>LLVM Developers mailing list
>>LLVMdev at cs.uiuc.edu         http://llvm.cs.uiuc.edu
>>http://mail.cs.uiuc.edu/mailman/listinfo/llvmdev
>>    
>>
>>------------------------------------------------------------------------
>>
>>_______________________________________________
>>LLVM Developers mailing list
>>LLVMdev at cs.uiuc.edu         http://llvm.cs.uiuc.edu
>>http://mail.cs.uiuc.edu/mailman/listinfo/llvmdev
>>    
>>

I'll see what I can do. However, I have to balance your need for "big
bang" change with Chris' fervor for incremental changes. Making a huge
change makes his review task harder. There are only seven files to merge
so at most seven change sets. I'll probably get these done early next
week. Sorry for the hassle, but I want to get lib/System simple,
complete and working so I can forget about it and move on. I'll be done
with this by the new year.

Reid.

On Fri, 2004-12-24 at 08:06, Jeff Cohen wrote:
> And I had just fixed VC++ so that it *did* handle the dependencies with 
> platform correctly...  This means that each file in System you change to 
> eliminate the #include from platform will require a corresponding change 
> to System.vproj.  I'd appreciate it if you got it over with all at
once,
> otherwise VC++ support will be broken more often than not.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 189 bytes
Desc: This is a digitally signed message part
URL:
<http://lists.llvm.org/pipermail/llvm-dev/attachments/20041224/4d840424/attachment.sig>