Skip to content

paket.template - file type doesn't respect dir without slash at the end #698

Description

@theimowski
type file
id xxx
version 1.0
authors xxx
description
        description of this test package
files
    bin/Debug/ ==> lib

bin/Debug doesn't work
bin/Debug/ works

/cc @mavnn

Activity

  1. forki commented on Mar 12, 2015

    @forki
    Member

    ok this one is my fault. I changed that to FAKE's globbing. and the behaviour is already in FAKE.

    Could you try to find a workaround around https://github.com/fsprojects/Paket/blob/master/src/Paket.Core/TemplateFile.fs#L328?

    we might even discuss changes to the globbing in FAKE
    // @colinbull

  2. added a commit that references this issue on Mar 12, 2015
    07a0b07
  3. added a commit that references this issue on Mar 12, 2015
    3b304ef
  4. forki commented on Mar 12, 2015

    @forki
    Member

    I wonder if it really copies the whole directory including subdirectories.

  5. theimowski commented on Mar 12, 2015

    @theimowski
    MemberAuthor

    In my case it did

  6. forki commented on Mar 12, 2015

    @forki
    Member

    Ok then I think it should go upstream into FAKE

  7. forki commented on Mar 12, 2015

    @forki
    Member

    It already annoyed some people there

  8. theimowski commented on Mar 12, 2015

    @theimowski
    MemberAuthor

    Go for It

  9. colinbull commented on Mar 12, 2015

    @colinbull
    Contributor

    There is a slight subtly here, when we have the trailing slash this allows the globbing implementation to figure out that it is a directory so something like bin\Debug\ will be turned into the following search tokens, [Directory(bin); Directory(Debug)]. Now if we don't have a the trailing slash we get [Directory(bin); Filename(Debug)], this is supports extension-less files. Now considering how rare they I could just test to see if the last token has an extension and emit the correct token. What do you think?

  10. forki commented on Mar 12, 2015

    @forki
    Member

    Please take a look at 07a0b07
    and fsprojects/FAKE@8f83457 - I thought the if Directory.Exists would handle this?

  11. colinbull commented on Mar 12, 2015

    @colinbull
    Contributor

    This causes a fair few test failures. I have added some failing tests and a fix colinbull/FAKE@e3eb041 sent you a PR.

  12. forki commented on Mar 12, 2015

    @forki
    Member

    yep. just reverted that thing. It's not correct. And the workarounf Paket will also fail on wildcards ;-(

  13. added a commit that references this issue on Mar 12, 2015
    ed4924d
  14. theimowski commented on Mar 12, 2015

    @theimowski
    MemberAuthor

    Should we revert then?

  15. forki commented on Mar 12, 2015

    @forki
    Member

    could you please try with 0.33.15 - it uses colin's stuff from FAKE

  16. theimowski commented on Mar 12, 2015

    @theimowski
    MemberAuthor

    works fine, thanks

  17. colinbull commented on Mar 12, 2015

    @colinbull
    Contributor

    Good too hear 👍

  18. forki commented on Mar 12, 2015

    @forki
    Member
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions