Patch Guidelines - and how to find something to work on

Discussion in 'Submit: code patches (MediaPortal/TV-Server/etc.)' started by infinite.loop, May 26, 2009.

  1. infinite.loop
    • Team MediaPortal

    infinite.loop Retired Team Member

    Joined:
    December 26, 2004
    Messages:
    16,163
    Likes Received:
    3,861
    Gender:
    Male
    Location:
    127.0.0.1
    Ratings:
    +4,154 / 7
    Home Country:
    Austria Austria
    Show System Specs
    Patch Guidelines - and how to find something to work on

    We appreciate every help we get from our community. Especially if we receive patches. :)
    If you are a developer and want to contribute to this project, then you might wonder what you could do.
    Your first stop should be our Mantis bug tracker. Or the roadmaps to be more specific.



    Roadmaps:
    1. MediaPortal 1
    2. TV-Server
    3. DeployTool / NSIS
    There you can find all the confirmed bugs and new features the team agreed to add.
    If an issue was not yet assigned to a developer, then you can give it a try if you want.
    If it is a time-consuming change, then it would be wise to start a work in progress thread, to avoid that someone else starts to work on the same issue.

    But to get a patch into our svn, a few criteria have to be met.
    This is why we have created a ::description in our wiki:: which explains the patch process from start to end in detail.
    Please read ::this description:: and follow it if you are going to submit a patch.
    :D
     
  2. Google AdSense Guest Advertisement



    to hide all adverts.
  3. arion_p
    • Team MediaPortal

    arion_p Retired Team Member

    Joined:
    February 7, 2007
    Messages:
    3,352
    Likes Received:
    1,447
    Occupation:
    Developer
    Location:
    Athens
    Ratings:
    +1,522 / 0
    Home Country:
    Greece Greece
    Show System Specs
    What about patches already submitted to the patch queue. Will they be considered or do we need to resubmit them here too?

    Also what about minor fixes (e.g. 1-2 lines of code) that have already been posted in the (old) bug reporting forums or the development forums?

    Thanks,

    Arion
     
  4. infinite.loop
    • Team MediaPortal

    infinite.loop Retired Team Member

    Joined:
    December 26, 2004
    Messages:
    16,163
    Likes Received:
    3,861
    Gender:
    Male
    Location:
    127.0.0.1
    Ratings:
    +4,154 / 7
    Home Country:
    Austria Austria
    Show System Specs
    reason why we now handle patches here is that on sourceforge, communication is a problem.
    also developers look more into the forums than on the SF page. but the allready submitted patches on SF do not have to be added here again.
    would cause some confusin to have them twice i think. :)
    Bugreportforums are definitly the wrong place to submit patches.
    you can ask an moderator to move threads from the "general development" forum to this one here. but make sure that you edit the first post to match the patch-guideline before doing so.
    Our devs request to only receive .patch files here.
    I do not know what will happen if you post the "plain code" for small changes you were talking about. :)
     
  5. tourettes
    • Team MediaPortal

    tourettes Retired Team Member

    Joined:
    January 7, 2005
    Messages:
    17,301
    Likes Received:
    4,595
    Ratings:
    +4,810 / 3
    I think it would be even better to create a new thread in this section and just point the "old" thread in bug forums as an additional information. Just to make sure that this forum part is not going to get cluttered with user comments etc. and thus making this section less interested for the developers.

    Patch creation is almost automatic with Tortoise, so no plain text patches should be posted. Same amount that the patch poster is saving when he/she doesn't create the .patch file is already going to be wasted few times when developer finds the correct file in VS :)
     
    • Like Like x 1
  6. tourettes
    • Team MediaPortal

    tourettes Retired Team Member

    Joined:
    January 7, 2005
    Messages:
    17,301
    Likes Received:
    4,595
    Ratings:
    +4,810 / 3
    Please note that following addition to the patch submission guidelines has been added:

    Keeping the documentation up to date is as important as the code improvements.
     
    • Like Like x 1
  7. Tony Wall
    • Premium Supporter

    Tony Wall MP Donator

    Joined:
    May 31, 2012
    Messages:
    18
    Likes Received:
    9
    Gender:
    Male
    Ratings:
    +14 / 0
    Home Country:
    United Kingdom United Kingdom
Loading...

Users Viewing Thread (Users: 0, Guests: 0)

  1. This site uses cookies to help personalise content, tailor your experience and to keep you logged in if you register.
    By continuing to use this site, you are consenting to our use of cookies.
    Dismiss Notice
  • About The Project

    The vision of the MediaPortal project is to create a free open source media centre application, which supports all advanced media centre functions, and is accessible to all Windows users.

    In reaching this goal we are working every day to make sure our software is one of the best.

             

  • Support MediaPortal!

    The team works very hard to make sure the community is running the best HTPC-software. We give away MediaPortal for free but hosting and software is not for us.

    Care to support our work with a few bucks? We'd really appreciate it!