January 16, 2012
On 2012-01-16 16:38, Andrei Alexandrescu wrote:
> On 1/16/12 3:02 AM, Jacob Carlborg wrote:
>> A large list of dependencies?? The only runtime dependency is zlib.
>
> That's in the case we want to include it in tools/.
>
> Andrei
>

So you're referring to the build dependencies.

* DSSS - It's possible to build it with rdmd instead. This will remove the dependency on DSSS, I assume rdmd is ok to have as a dependency.

* D1 - I plan to port the project to D2 (using Tango)

* Mercurial - I plan to move the project go github

That changes the dependencies to:

* A D2 compiler
* Tango
* zlib
* Git

Of those dependencies I guess it's only Tango that would be a blocker.

-- 
/Jacob Carlborg
January 16, 2012
On 1/16/12 9:22 AM, Jesse Phillips wrote:
> With that change it is likely you'll want to change update to use
> upstream instead of origin. Though it is probably dependent on if the
> user has upstream which I don't know of a good way to check. Or you
> could just require those to have an upstream.
>
> https://github.com/D-Programming-Language/tools/blob/c5c61a67b7f043f8127d3bf92c98d19a30d6b06d/update.sh#L136
>
>
> You can add the remote to your repository with
>
> git remote add upstream git@github.com:D-Programming-Language/dmd.git

At what point in the script should I insert that command?

Andrei
January 16, 2012
On 1/16/12 5:46 AM, Nick Sabalausky wrote:
>> I don't mean to say that your tool is bad or that you shouldn't have made
>> it, but the issues you addressed with DVM can be fixed, and really, I've
>> been detecting a bit of NIH coming from your general direction. I've
>> noticed that any time you want something, you go off and do it yourself,
>> and to hell with any part of the community that's been working on anything
>> similar. If you see a problem with the existing options, at the very least
>> you could say something instead of ignoring it and going cowboy-coder,
>> running off undermining anything that doesn't have the official
>> "Championed by Andrei" stamp.
>>
>
> I apologize for the tone, I overreacted. I'm sure you weren't trying to
> undermine anyone or anything, just posting a potentially helpful tool, like
> we all try to do. I do think it would be nice for you to give feedback if
> there's any D tools/libs that are similar to what you want but don't quite
> fit the bill for one reason or another. But I understand if it wasn't clear
> that DVM was trying to move in that direction.

As always, constructive feedback is welcome.

In this case I don't see a reason to read much into it. The script simply puts together the actions that someone working on dmd need to do needs to type on a regular basis, and parallelizes the slowest parts of it. It's not a "tool" and not pretending to be one - it's a very specific collection of actions. Quite a textbook example of the usefulness of Unix scripting.

I wasn't enumerating "issues" that need to be "fixed" with DVM - merely mentioning simple reasons that make DVM and update.sh different enough to not compete.


Andrei


January 16, 2012
On Monday, 16 January 2012 at 15:48:17 UTC, Jacob Carlborg wrote:
> On 2012-01-16 16:38, Andrei Alexandrescu wrote:
>> On 1/16/12 3:02 AM, Jacob Carlborg wrote:
>>> A large list of dependencies?? The only runtime dependency is zlib.
>>
>> That's in the case we want to include it in tools/.
>>
>> Andrei
>>
>
> So you're referring to the build dependencies.
>
> * DSSS - It's possible to build it with rdmd instead. This will remove the dependency on DSSS, I assume rdmd is ok to have as a dependency.
>
> * D1 - I plan to port the project to D2 (using Tango)
>
> * Mercurial - I plan to move the project go github
>
> That changes the dependencies to:
>
> * A D2 compiler
> * Tango
> * zlib
> * Git
>
> Of those dependencies I guess it's only Tango that would be a blocker.

Wouldn't it be weird to have a dependency on Tango when the std lib is Phobos?

/Jonas



January 16, 2012
On 2012-01-16 21:24, jdrewsen wrote:
> On Monday, 16 January 2012 at 15:48:17 UTC, Jacob Carlborg wrote:
>> On 2012-01-16 16:38, Andrei Alexandrescu wrote:
>>> On 1/16/12 3:02 AM, Jacob Carlborg wrote:
>>>> A large list of dependencies?? The only runtime dependency is zlib.
>>>
>>> That's in the case we want to include it in tools/.
>>>
>>> Andrei
>>>
>>
>> So you're referring to the build dependencies.
>>
>> * DSSS - It's possible to build it with rdmd instead. This will remove
>> the dependency on DSSS, I assume rdmd is ok to have as a dependency.
>>
>> * D1 - I plan to port the project to D2 (using Tango)
>>
>> * Mercurial - I plan to move the project go github
>>
>> That changes the dependencies to:
>>
>> * A D2 compiler
>> * Tango
>> * zlib
>> * Git
>>
>> Of those dependencies I guess it's only Tango that would be a blocker.
>
> Wouldn't it be weird to have a dependency on Tango when the std lib is
> Phobos?
>
> /Jonas

I haven't said I want to include it in the "tools" repository, that was what Andrei said.

-- 
/Jacob Carlborg
January 17, 2012
On Monday, 16 January 2012 at 16:57:48 UTC, Andrei Alexandrescu wrote:
> On 1/16/12 9:22 AM, Jesse Phillips wrote:
>> With that change it is likely you'll want to change update to use
>> upstream instead of origin. Though it is probably dependent on if the
>> user has upstream which I don't know of a good way to check. Or you
>> could just require those to have an upstream.
>>
>> https://github.com/D-Programming-Language/tools/blob/c5c61a67b7f043f8127d3bf92c98d19a30d6b06d/update.sh#L136
>>
>>
>> You can add the remote to your repository with
>>
>> git remote add upstream git@github.com:D-Programming-Language/dmd.git
>
> At what point in the script should I insert that command?
>
> Andrei

Sorry, I meant that you could add the remote repository "upstream" to your personal git repository, the one you, Andrei cloned on your system. The change I am suggesting is to change line 136 and 137 so that

-            git pull origin master && \
-            git pull origin master --tags && \
+            git pull upstream master --tags && \

just noticed you are pulling twice, when you can do it once.
1 2 3
Next ›   Last »