www.digitalmars.com         C & C++   DMDScript  

digitalmars.D.bugs - [Issue 9269] New: [2.061] rdmd -unittest fails with lcurl not found

reply d-bugmail puremagic.com writes:
http://d.puremagic.com/issues/show_bug.cgi?id=9269

           Summary: [2.061] rdmd -unittest fails with lcurl not found
           Product: D
           Version: D2
          Platform: All
        OS/Version: All
            Status: NEW
          Severity: normal
          Priority: P2
         Component: Phobos
        AssignedTo: nobody puremagic.com
        ReportedBy: philippe.sigaud gmail.com



16:09:57 CET ---
DMD 2.061:

A std.traits unit test has a pragma(lib, "curl") declaration. That makes

$ rdmd -unittest myfile.d

fail because dmd -deps (called by rdmd) activate the pragma and if the user has
no curl installed, this stops the compilation.

Btw,

$ dmd -unittest myfile.d

works OK.

The concerned std.traits unit test should be changed.

-- 
Configure issuemail: http://d.puremagic.com/issues/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
Jan 04 2013
next sibling parent d-bugmail puremagic.com writes:
http://d.puremagic.com/issues/show_bug.cgi?id=9269


Andrej Mitrovic <andrej.mitrovich gmail.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |andrej.mitrovich gmail.com



08:08:41 PST ---
*** Issue 9007 has been marked as a duplicate of this issue. ***

-- 
Configure issuemail: http://d.puremagic.com/issues/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
Jan 04 2013
prev sibling next sibling parent d-bugmail puremagic.com writes:
http://d.puremagic.com/issues/show_bug.cgi?id=9269


Andrej Mitrovic <andrej.mitrovich gmail.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |pisetta.gianni alice.it



10:19:57 PST ---
*** Issue 9349 has been marked as a duplicate of this issue. ***

-- 
Configure issuemail: http://d.puremagic.com/issues/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
Jan 18 2013
prev sibling next sibling parent d-bugmail puremagic.com writes:
http://d.puremagic.com/issues/show_bug.cgi?id=9269




11:16:57 PST ---
Created an attachment (id=1181)
patch to std.traits.d file for eliminating the dependency over libcurl

Patch that solves the issue of libcurl needed in the unittest of std.traits
module replacing the tests with the core.sync.barrier module.

-- 
Configure issuemail: http://d.puremagic.com/issues/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
Jan 21 2013
prev sibling next sibling parent d-bugmail puremagic.com writes:
http://d.puremagic.com/issues/show_bug.cgi?id=9269




11:22:41 PST ---
I tought that it is incorrect to use a module that needs an external library,
so i replaced the use of etc.c.curl module with the core.sync.barrier module,
that doesn't need other libraries and i think it is unlikely that that module
is changed in the future.
I have attached the patch file generated with diff.

Gianni Pisetta

-- 
Configure issuemail: http://d.puremagic.com/issues/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
Jan 21 2013
prev sibling next sibling parent d-bugmail puremagic.com writes:
http://d.puremagic.com/issues/show_bug.cgi?id=9269




11:35:45 PST ---

 Created an attachment (id=1181) [details]
 patch to std.traits.d file for eliminating the dependency over libcurl
 
 Patch that solves the issue of libcurl needed in the unittest of std.traits
 module replacing the tests with the core.sync.barrier module.
Thanks. Do you want to make a pull request or should we do it for you? -- Configure issuemail: http://d.puremagic.com/issues/userprefs.cgi?tab=email ------- You are receiving this mail because: -------
Jan 21 2013
prev sibling next sibling parent d-bugmail puremagic.com writes:
http://d.puremagic.com/issues/show_bug.cgi?id=9269




17:27:00 PST ---
https://github.com/D-Programming-Language/phobos/pull/1088

Thanks again Gianni.

-- 
Configure issuemail: http://d.puremagic.com/issues/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
Jan 21 2013
prev sibling next sibling parent d-bugmail puremagic.com writes:
http://d.puremagic.com/issues/show_bug.cgi?id=9269




Commits pushed to master at https://github.com/D-Programming-Language/phobos

https://github.com/D-Programming-Language/phobos/commit/40d9ebcdfb061856ae89ecbc8c1b723b4d63d8f3
Fixes Issue 9269 - Remove import to curl to avoid unittest build failure.

https://github.com/D-Programming-Language/phobos/commit/1ec2e521b2fb328da8faddfa1e0edd940ed2cf5c


Issue 9269 - Remove import to curl to avoid unittest build failure

-- 
Configure issuemail: http://d.puremagic.com/issues/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
Jan 21 2013
prev sibling next sibling parent d-bugmail puremagic.com writes:
http://d.puremagic.com/issues/show_bug.cgi?id=9269


yebblies <yebblies gmail.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |yebblies gmail.com



Fixed?

-- 
Configure issuemail: http://d.puremagic.com/issues/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
Jan 21 2013
prev sibling parent d-bugmail puremagic.com writes:
http://d.puremagic.com/issues/show_bug.cgi?id=9269


Andrej Mitrovic <andrej.mitrovich gmail.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |RESOLVED
         Resolution|                            |WORKSFORME



15:29:08 PST ---
Should be fixed. Reopen if necessary.

-- 
Configure issuemail: http://d.puremagic.com/issues/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
Jan 25 2013