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.

Sign up for GitHub

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

Jump to bottom

move workspace to writable directory (windows 7 x64) [+1] #2847

Closed
danrubel opened this issue May 1, 2012 · 12 comments
Closed

move workspace to writable directory (windows 7 x64) [+1] #2847

danrubel opened this issue May 1, 2012 · 12 comments
Assignees
@devoncarew
Labels
closed-obsolete Closed as the reported issue is no longer relevant os-windows
Milestone
M3

Comments

@danrubel
Copy link

danrubel commented May 1, 2012

[user feedback]
on windows 7 x64 installs

editor fails to load due to not have -data permissions

when corrected with the -data parameter samples silently fail to load with no obvious cause

relocating the entire dart unzip to a directory other than Program Files with write permissions clears both issues

probably a significant hurdle to people new at programming on windows
////////////////////////////////////////////////////////////////////////////////////
Editor Version: 6167
@danrubel
Copy link
Author

danrubel commented May 1, 2012

Added OpSys-Windows label.

@clayberg
Copy link

clayberg commented May 2, 2012

This should be solved by a install doc change. This is a zip install, so the user can put it anywhere. They should not unzip it to the Program Files directory. If the user is an expert, they could also edit the .ini file to point to a specific workspace.

Set owner to @danrubel.

@clayberg
Copy link

clayberg commented May 7, 2012

[user feedback]

Following from:
Issue #2847: move workspace to writable directory (windows 7 x64)

Can the workspace and current project/directory load configuration possibly be moved out of the Dart instal directory (wherever this is put)
I try and work with the latest Dart version (which I guess is beneficial to everyone), but it is discouraging having to recreate my preferred layout and directory load from scratch every time I copy a later Dart build.
Having an ini file or other means of pointing to user config data which could be copied without any other risk of poluting the new build would be really helpful.

////////////////////////////////////////////////////////////////////////////////////
Editor Version: 7209

Changed the title to: "move workspace to writable directory (windows 7 x64) [+1]".

@danrubel
Copy link
Author

danrubel commented Jun 5, 2012

Added this to the M1 milestone.

@clayberg
Copy link

Issue #4458 has been merged into this issue.

@clayberg
Copy link

Removed this from the M1 milestone.
Added this to the Later milestone.

@devoncarew
Copy link
Member

We're now defaulting to putting the workspace directory in the user's home directory. On Windows, this is user.home/DartEditor. On Linux, it's user.home/.dartEditor. On the mac it's user.home/Libraries/Application Support/DartEditor.

This should address part of this issue. I think we're still making modifications in the configuration directory however.

@clayberg
Copy link

Removed this from the Later milestone.
Added this to the M2 milestone.

@danrubel
Copy link
Author

danrubel commented Nov 9, 2012

Removed this from the M2 milestone.
Added this to the M3 milestone.

@danrubel
Copy link
Author

Tackle this when you can.

Set owner to @devoncarew.
Removed this from the M3 milestone.
Added this to the M4 milestone.

@devoncarew
Copy link
Member

Closing this one out. We do create the workspace in a writable dir - c:\Users\user\DartEditor. And for now, the editor app should not be placed in the Program Files directory.

Added AssumedStale label.

@clayberg
Copy link

Removed this from the M4 milestone.
Added this to the M3 milestone.

@danrubel danrubel added Type-Defect os-windows closed-obsolete Closed as the reported issue is no longer relevant labels Feb 16, 2013
@danrubel danrubel added this to the M3 milestone Feb 16, 2013
dart-bot pushed a commit that referenced this issue Mar 23, 2021
@sigurdm
Bump pub
633d4ac 
New commits included:
```
> git log --format="%C(auto) %h %s" 0e657414a472e74ca5dd76ae0db50cc060251dec..255a3091fc278b04be74d246a3bec8743ef4d0b7
 255a3091 Vendor package:tar and package:chunked_stream (#2932)
 86bf8b20 Handle relative git-url-paths correctly when --directory (#2919)
 3716a681 Let `pub add` fail if extra arguments are passed (#2927)
 a03ac729 Minor cleanup to reduce risk of using path.current (#2924)
 e87b7b66 Added null check for name in UserInfo class (#2918)
 056a8c9a pub deps --json (#2896)
 53a69e27 Fix .packages entries of relative path deps when using --directory (#2916)
 d6308efc pub upgrade command shows count of discontinued packages (#2908)
 51744805 Upgrade to the null safe versions of all dependencies (#2913)
 e0d538c7 Introduce .pubignore (#2787)
 79f3a8b9 pub outdated: added clear message when no outdated packages. (#2898)
 22463872 `cache clean` (#2904)
 11e7b2ce `publish --dry-run` informs that the server might do more checks (#2883)
 b6977d50 Remove untrue assert (#2884)
 35841f8d Merge branch 'cherry_picks_for_2_12'
 0db3255b Don't fail on failed status listing (#2877)
 53e8ecca Don't allow outdated taking arguments (#2872)
 e83a1dc1 Enable asserts when testing pub (#2754)
 178f2edb Add --directory option (#2876)
 5aadb70e Don't fail on failed status listing (#2877)
 4bf8a927 Remove unused field (#2878)
 73ad5426 Don't allow outdated taking arguments (#2872)
 9a70949e Use Dart library to read and write tar files (#2817)
 2f74230c Do not recommend decativating packages (#2871)
 b1697a27 Use full error message string in CommandResolutionFailedException (#2870)
 16a6210d Upgrade `downgrade --help`: `downgrade` actually updates `pubspec.lock` (#2859)
 6e240ea9 Use cached version listings as heuristic when prefetching (#2851)
 58152f7c Allow trailing slash in PUB_HOSTED_URL (#2856)
 b1bf9a33 Handle poor package-listing responses robustly. (#2847)
 d941bd24 Fix request metadata when overriding dependencyType (#2848)
 ```

Change-Id: Id6ed5698330fc0cdb507e024eabc34f925ca9208
Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/192304
Commit-Queue: Sigurd Meldgaard <[email protected]>
Reviewed-by: Jonas Jensen <[email protected]>
dart-bot pushed a commit that referenced this issue Mar 24, 2021
@athomas
Revert "Bump pub"
424c554 
This reverts commit 633d4ac.

Reason for revert: breaks dependency resolution in some cases.

Original change's description:
> Bump pub
>
> New commits included:
> ```
> > git log --format="%C(auto) %h %s" 0e657414a472e74ca5dd76ae0db50cc060251dec..255a3091fc278b04be74d246a3bec8743ef4d0b7
>  255a3091 Vendor package:tar and package:chunked_stream (#2932)
>  86bf8b20 Handle relative git-url-paths correctly when --directory (#2919)
>  3716a681 Let `pub add` fail if extra arguments are passed (#2927)
>  a03ac729 Minor cleanup to reduce risk of using path.current (#2924)
>  e87b7b66 Added null check for name in UserInfo class (#2918)
>  056a8c9a pub deps --json (#2896)
>  53a69e27 Fix .packages entries of relative path deps when using --directory (#2916)
>  d6308efc pub upgrade command shows count of discontinued packages (#2908)
>  51744805 Upgrade to the null safe versions of all dependencies (#2913)
>  e0d538c7 Introduce .pubignore (#2787)
>  79f3a8b9 pub outdated: added clear message when no outdated packages. (#2898)
>  22463872 `cache clean` (#2904)
>  11e7b2ce `publish --dry-run` informs that the server might do more checks (#2883)
>  b6977d50 Remove untrue assert (#2884)
>  35841f8d Merge branch 'cherry_picks_for_2_12'
>  0db3255b Don't fail on failed status listing (#2877)
>  53e8ecca Don't allow outdated taking arguments (#2872)
>  e83a1dc1 Enable asserts when testing pub (#2754)
>  178f2edb Add --directory option (#2876)
>  5aadb70e Don't fail on failed status listing (#2877)
>  4bf8a927 Remove unused field (#2878)
>  73ad5426 Don't allow outdated taking arguments (#2872)
>  9a70949e Use Dart library to read and write tar files (#2817)
>  2f74230c Do not recommend decativating packages (#2871)
>  b1697a27 Use full error message string in CommandResolutionFailedException (#2870)
>  16a6210d Upgrade `downgrade --help`: `downgrade` actually updates `pubspec.lock` (#2859)
>  6e240ea9 Use cached version listings as heuristic when prefetching (#2851)
>  58152f7c Allow trailing slash in PUB_HOSTED_URL (#2856)
>  b1bf9a33 Handle poor package-listing responses robustly. (#2847)
>  d941bd24 Fix request metadata when overriding dependencyType (#2848)
>  ```
>
> Change-Id: Id6ed5698330fc0cdb507e024eabc34f925ca9208
> Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/192304
> Commit-Queue: Sigurd Meldgaard <[email protected]>
> Reviewed-by: Jonas Jensen <[email protected]>

[email protected],[email protected]

Change-Id: I364a26468a36438bbbc887e613241030decef4d3
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/192748
Reviewed-by: Alexander Thomas <[email protected]>
Commit-Queue: Alexander Thomas <[email protected]>
dart-bot pushed a commit that referenced this issue May 11, 2021
@jonasfj
Bump pub
53448de 
New commits include:
```
$ git log --format="%C(auto) %h %s" 0e657414a472e74ca5dd76ae0db50cc060251dec..00c00e8adf9706bebe8f94483b7663c5f36f59d2
 00c00e8a Vendor tar (#2987)
 291705ca Being gradual migration to null-safety (#2988)
 c5f52a37 Fix CI (#2989)
 74040a45 Update to analyzer 1.5.0, migrate from deprecated AnalysisSession.getParsedUnit() (#2975)
 ce951d70 Fix dry-run tests for publishing  by ensuring there is a server to reject requests (#2978)
 018c9650 Update LICENSE (#2944)
 2614f15c Revert "Vendor package:tar and package:chunked_stream (#2932)" (#2940)
 12d9f457 Handle package:tar cancellations
 255a3091 Vendor package:tar and package:chunked_stream (#2932)
 86bf8b20 Handle relative git-url-paths correctly when --directory (#2919)
 3716a681 Let `pub add` fail if extra arguments are passed (#2927)
 a03ac729 Minor cleanup to reduce risk of using path.current (#2924)
 e87b7b66 Added null check for name in UserInfo class (#2918)
 056a8c9a pub deps --json (#2896)
 53a69e27 Fix .packages entries of relative path deps when using --directory (#2916)
 d6308efc pub upgrade command shows count of discontinued packages (#2908)
 51744805 Upgrade to the null safe versions of all dependencies (#2913)
 e0d538c7 Introduce .pubignore (#2787)
 79f3a8b9 pub outdated: added clear message when no outdated packages. (#2898)
 22463872 `cache clean` (#2904)
 11e7b2ce `publish --dry-run` informs that the server might do more checks (#2883)
 b6977d50 Remove untrue assert (#2884)
 35841f8d Merge branch 'cherry_picks_for_2_12'
 0db3255b Don't fail on failed status listing (#2877)
 53e8ecca Don't allow outdated taking arguments (#2872)
 e83a1dc1 Enable asserts when testing pub (#2754)
 178f2edb Add --directory option (#2876)
 5aadb70e Don't fail on failed status listing (#2877)
 4bf8a927 Remove unused field (#2878)
 73ad5426 Don't allow outdated taking arguments (#2872)
 9a70949e Use Dart library to read and write tar files (#2817)
 2f74230c Do not recommend decativating packages (#2871)
 b1697a27 Use full error message string in CommandResolutionFailedException (#2870)
 16a6210d Upgrade `downgrade --help`: `downgrade` actually updates `pubspec.lock` (#2859)
 6e240ea9 Use cached version listings as heuristic when prefetching (#2851)
 58152f7c Allow trailing slash in PUB_HOSTED_URL (#2856)
 b1bf9a33 Handle poor package-listing responses robustly. (#2847)
 d941bd24 Fix request metadata when overriding dependencyType (#2848)
```

Change-Id: Id7cc4c09e74c02a92bcafe1a9d9bab9431900540
Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/199040
Reviewed-by: Alexander Thomas <[email protected]>
Commit-Queue: Jonas Jensen <[email protected]>
This issue was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@devoncarew devoncarew

Labels
closed-obsolete Closed as the reported issue is no longer relevant os-windows
Projects
None yet
Milestone
M3
Development

No branches or pull requests
3 participants
@devoncarew @danrubel @clayberg