Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 0.1
    • Fix Version/s: 0.1
    • Component/s: Perl - Compiler
    • Labels:
      None
    • Patch Info:
      Patch Available

      Description

      I just had a look at the Perl stuff generated from tutorial.thrift:

      ....tutorial/gen-perl/
      ........shared/
      ............Constants.pm
      ............Types.pm
      ........tutorial/
      ............Constants.pm
      ............Types.pm
      ........Calculator.pm
      ........SharedService.pm

      Since 'Calculator' (resp. 'SharedService') is defined in 'tutorial.thrift'
      (resp. 'shared.thrift'), whose namespace declaration is 'tutorial' (resp.
      'shared'), I would have expected the following:

      ....tutorial/gen-perl/
      ........shared/
      ............Constants.pm
      ............Types.pm
      ............SharedService.pm
      ........tutorial/
      ............Constants.pm
      ............Types.pm
      ............Calculator.pm

      (Which is what you get for Python, by the way)

      With the current layout, if I want to define a service called 'Calculator'
      under my own namespace 'foo', I would get:

      ....foo/
      ........Constants.pm
      ........Types.pm
      ....Calculator.pm

      ,which would clash with Calculator.pm generated from tutorial.thrift.

      1. perl-namespaces.diff
        8 kB
        Carlos Valiente

        Activity

          People

          • Assignee:
            T Jake Luciani
            Reporter:
            Carlos Valiente
          • Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development